[apache#6133] Improvement(core): Supports get Fileset schema location…

… in the AuthorizationUtils use StringUtils replace ==null.
Abyss-lord · Jan 13, 2025 · 1358bae · 1358bae
1 parent accfc59
commit 1358bae
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/core/src/main/java/org/apache/gravitino/authorization/AuthorizationUtils.java b/core/src/main/java/org/apache/gravitino/authorization/AuthorizationUtils.java
@@ -521,7 +521,7 @@ public static List<String> getMetadataObjectLocation(
                           nameIdentifier -> {
                             Fileset fileset = filesetCatalog.loadFileset(nameIdentifier);
                             String filesetLocation = fileset.storageLocation();
-                            if (filesetLocation != null && !filesetLocation.isEmpty())
+                            if (!StringUtils.isNotBlank(filesetLocation))
                               locations.add(filesetLocation);
                           });
                 }
@@ -545,7 +545,7 @@ public static List<String> getMetadataObjectLocation(
               Table table = GravitinoEnv.getInstance().tableDispatcher().loadTable(ident);
               if (table.properties().containsKey(HiveConstants.LOCATION)) {
                 String tableLocation = table.properties().get(HiveConstants.LOCATION);
-                if (tableLocation != null && tableLocation.isEmpty()) {
+                if (StringUtils.isNotBlank(tableLocation)) {
                   locations.add(tableLocation);
                 } else {
                   LOG.warn("Table %s location is not found", ident);