[BugFix] Fix NullPointerException in CatalogMgr.java (StarRocks#52501)

Signed-off-by: immutablecodes <[email protected]> Signed-off-by: immutablecodes <[email protected]> Co-authored-by: alvin <[email protected]> Signed-off-by: zhiminr.ren <[email protected]>
renzhimin7 · Nov 7, 2024 · 0a2cc70 · 0a2cc70
1 parent 208ef2c
commit 0a2cc70
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/fe/fe-core/src/main/java/com/starrocks/server/CatalogMgr.java b/fe/fe-core/src/main/java/com/starrocks/server/CatalogMgr.java
@@ -191,7 +191,7 @@ public void alterCatalog(AlterCatalogStmt stmt) {
                 Map<String, String> properties = ((ModifyTablePropertiesClause) stmt.getAlterClause()).getProperties();
                 String serviceName = properties.get("ranger.plugin.hive.service.name");
 
-                if (serviceName.isEmpty()) {
+                if (Strings.isNullOrEmpty(serviceName)) {
                     if (Config.access_control.equals("ranger")) {
                         Authorizer.getInstance().setAccessControl(catalogName, new RangerStarRocksAccessController());
                     } else {
@@ -340,7 +340,7 @@ public void replayAlterCatalog(AlterCatalogLog log) {
             String catalogName = log.getCatalogName();
             Map<String, String> properties = log.getProperties();
             String serviceName = properties.get("ranger.plugin.hive.service.name");
-            if (serviceName.isEmpty()) {
+            if (Strings.isNullOrEmpty(serviceName)) {
                 if (Config.access_control.equals("ranger")) {
                     Authorizer.getInstance().setAccessControl(catalogName, new RangerStarRocksAccessController());
                 } else {