HIVE-28898: using get_table_meta instead of getMS().getTableMetas()

HIVE-28898: remove condition

Author: jintongjiang

ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/metastore/filtercontext/TableFilterContext.java

@@ -81,9 +81,7 @@ public HiveMetaStoreAuthzInfo getAuthzContext() {
   }
 
   private List<HivePrivilegeObject> getInputHObjs() {
-    if (LOG.isDebugEnabled()) {
-      LOG.debug("==> TableFilterContext.getOutputHObjs()");
-    }
+    LOG.debug("==> TableFilterContext.getOutputHObjs()");
 
     List<HivePrivilegeObject> ret = new ArrayList<>();
 
@@ -101,9 +99,7 @@ private List<HivePrivilegeObject> getInputHObjs() {
       }
     }
 
-    if (LOG.isDebugEnabled()) {
-      LOG.debug("<== TableFilterContext.getOutputHObjs(): ret=" + ret);
-    }
+    LOG.debug("<== TableFilterContext.getOutputHObjs(): ret=" + ret);
 
     return ret;
   }

standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java

@@ -6453,12 +6453,14 @@ public List<String> get_all_tables(final String dbname) throws MetaException {
     } catch (Exception e) { /* ignore */ }
 
     try {
-      List<TableMeta> filteredTableMetas = getMS().getTableMeta(parsedDbName[CAT_NAME], parsedDbName[DB_NAME], "*", null);
-      filteredTableMetas = FilterUtils.filterTableMetasIfEnabled(isServerFilterEnabled, filterHook, filteredTableMetas);
-      if (filteredTableMetas.isEmpty()) {
-        return new ArrayList<>();
+      if (getIfServerFilterenabled()) {
+        List<TableMeta> filteredTableMetas = get_table_meta(dbname, "*", null);
+        ret = filteredTableMetas.stream()
+            .map(TableMeta::getTableName)
+            .collect(Collectors.toList());
+      } else {
+        ret = getMS().getAllTables(parsedDbName[CAT_NAME], parsedDbName[DB_NAME]);
       }
-      ret = filteredTableMetas.stream().map(TableMeta::getTableName).collect(Collectors.toList());
     } catch (Exception e) {
       ex = e;
       throw newMetaException(e);