improve variable name

Author: loserwang1024

fluss-flink/fluss-flink-common/src/main/java/com/alibaba/fluss/flink/procedure/AclProcedure.java

@@ -48,7 +48,7 @@ public String identifier() {
      * through the Flink SQL procedure interface. It delegates execution to the internalCall method.
      *
      * @param context The procedure context used for execution environment and resource access.
-     * @param aclType Type of ACL operation to perform. Valid values are:
+     * @param action Actoion of ACL operation to perform. Valid values are:
      *     <ul>
      *       <li>{@code ADD}: Adds an ACL entry.
      *       <li>{@code DROP}: Removes an ACL entry.
@@ -62,7 +62,7 @@ public String identifier() {
      *       <li>{@code fluss-cluster.db_name.table_name} - table level
      *     </ul>
      *
-     * @param permissionType Permission type to grant or revoke. Common values include in {@link
+     * @param permission Permission type to grant or revoke. Common values include in {@link
      *     PermissionType}.
      * @param principal Principal (user or role) to apply the ACL to. Accepts:
      *     <ul>
@@ -85,23 +85,23 @@ public String identifier() {
      */
     public String[] call(
             ProcedureContext context,
-            String aclType,
+            String action,
             String resource,
-            String permissionType,
+            String permission,
             String principal,
             String operation)
             throws ExecutionException, InterruptedException {
-        return internalCall(aclType, resource, permissionType, principal, operation);
+        return internalCall(action, resource, permission, principal, operation);
     }
 
     private String[] internalCall(
-            String aclType,
+            String action,
             String resource,
             String permissionType,
             String principal,
             String operation)
             throws ExecutionException, InterruptedException {
-        AclType type = AclType.valueOf(aclType);
+        Action type = Action.valueOf(action);
         PermissionType permission = PermissionType.valueOf(permissionType);
         FlussPrincipal flussPrincipal = parsePrincipal(principal);
         OperationType operationType = OperationType.valueOf(operation);
@@ -222,7 +222,7 @@ private Resource parseResource(String resourceStr) {
         return resource;
     }
 
-    private enum AclType {
+    private enum Action {
         ADD,
         LIST,
         DROP

fluss-flink/fluss-flink-common/src/test/java/com/alibaba/fluss/flink/security/acl/FlinkAuthorizationITCase.java

@@ -414,12 +414,10 @@ List<String> listAcl(Resource resource, OperationType operationType) {
         TableResult tableResult =
                 tEnv.executeSql(
                         String.format(
-                                "CALL %s.sys.acl('LIST', '%s', 'ALLOW', '%s', '%s')",
+                                "CALL %s.sys.acl( action => 'LIST', resource => '%s', permission => 'ALLOW', principal => '%s', operation  => '%s')",
                                 ADMIN_CATALOG_NAME,
                                 getProcedureResourceString(resource),
-                                ResourceType.ANY == resource.getType()
-                                        ? "ANY"
-                                        : String.format("%s:%s", guest.getType(), guest.getName()),
+                                String.format("%s:%s", guest.getType(), guest.getName()),
                                 operationType.name()));
         List<Row> rows = CollectionUtil.iteratorToList(tableResult.collect());
         return rows.stream().map(Row::toString).collect(Collectors.toList());
@@ -432,9 +430,7 @@ void dropAcl(Resource resource, OperationType operationType)
                                 "CALL %s.sys.acl('DROP', '%s', 'ANY', '%s', '%s')",
                                 ADMIN_CATALOG_NAME,
                                 getProcedureResourceString(resource),
-                                ResourceType.ANY == resource.getType()
-                                        ? "ANY"
-                                        : String.format("%s:%s", guest.getType(), guest.getName()),
+                                String.format("%s:%s", guest.getType(), guest.getName()),
                                 operationType.name()))
                 .await();
     }