address comment

Author: jerqi

authorizations/authorization-common/src/main/java/org/apache/gravitino/authorization/common/RangerAuthorizationProperties.java

@@ -23,6 +23,8 @@
 
 /** The properties for Ranger authorization plugin. */
 public class RangerAuthorizationProperties extends AuthorizationProperties {
+  public static final String RANGER_PREFIX = "authorization.ranger";
+
   /** Ranger admin web URIs */
   public static final String RANGER_ADMIN_URL = "authorization.ranger.admin.url";
 
@@ -73,7 +75,7 @@ public RangerAuthorizationProperties(Map<String, String> properties) {
 
   @Override
   public String getPropertiesPrefix() {
-    return "authorization.ranger";
+    return RANGER_PREFIX;
   }
 
   @Override

authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java

@@ -84,7 +84,6 @@ public abstract class RangerAuthorizationPlugin
   protected RangerClientExtension rangerClient;
   protected RangerHelper rangerHelper;
   @VisibleForTesting public final String rangerAdminName;
-  private final int prefixLength;
 
   protected RangerAuthorizationPlugin(String metalake, Map<String, String> config) {
     this.metalake = metalake;
@@ -103,9 +102,6 @@ protected RangerAuthorizationPlugin(String metalake, Map<String, String> config)
     rangerServiceName = config.get(RangerAuthorizationProperties.RANGER_SERVICE_NAME);
     rangerClient = new RangerClientExtension(rangerUrl, authType, rangerAdminName, password);
 
-    // We should consider `.` to add 1
-    prefixLength = rangerAuthorizationProperties.getPropertiesPrefix().length() + 1;
-
     if (Boolean.parseBoolean(
         config.get(RangerAuthorizationProperties.RANGER_SERVICE_CREATE_IF_ABSENT))) {
       createRangerServiceIfNecessary(config, rangerServiceName);
@@ -1016,7 +1012,8 @@ protected String getConfValue(Map<String, String> conf, String key, String defau
   protected abstract Map<String, String> getServiceConfigs(Map<String, String> config);
 
   protected int getPrefixLength() {
-    return prefixLength;
+    // We should consider `.`. We need to add 1
+    return RangerAuthorizationProperties.RANGER_PREFIX.length() + 1;
   }
 
   @Override