rename

apache · Oct 30, 2024 · d9b9ed0 · d9b9ed0
1 parent 442bdb9
commit d9b9ed0
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 12 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
@@ -272,7 +272,7 @@ private static void callAuthorizationPluginImpl(
     }
   }
 
-  public static void removeAuthorizationPluginPrivileges(
+  public static void authorizationPluginRemovePrivileges(
       NameIdentifier ident, Entity.EntityType type) {
     // If we enable authorization, we should remove the privileges about the entity in the
     // authorization plugin.
@@ -288,7 +288,7 @@ public static void removeAuthorizationPluginPrivileges(
     }
   }
 
-  public static void renameAuthorizationPluginPrivileges(
+  public static void authorizationPluginRenamePrivileges(
       NameIdentifier ident, Entity.EntityType type, String newName) {
     // If we enable authorization, we should rename the privileges about the entity in the
     // authorization plugin.

diff --git a/core/src/main/java/org/apache/gravitino/hook/CatalogHookDispatcher.java b/core/src/main/java/org/apache/gravitino/hook/CatalogHookDispatcher.java
@@ -112,22 +112,22 @@ public Catalog alterCatalog(NameIdentifier ident, CatalogChange... changes)
       }
     }
     if (lastRenameChange != null) {
-      AuthorizationUtils.renameAuthorizationPluginPrivileges(
+      AuthorizationUtils.authorizationPluginRenamePrivileges(
           ident, Entity.EntityType.CATALOG, lastRenameChange.getNewName());
     }
     return alteredCatalog;
   }
 
   @Override
   public boolean dropCatalog(NameIdentifier ident) {
-    AuthorizationUtils.removeAuthorizationPluginPrivileges(ident, Entity.EntityType.CATALOG);
+    AuthorizationUtils.authorizationPluginRemovePrivileges(ident, Entity.EntityType.CATALOG);
     return dispatcher.dropCatalog(ident);
   }
 
   @Override
   public boolean dropCatalog(NameIdentifier ident, boolean force)
       throws NonEmptyEntityException, CatalogInUseException {
-    AuthorizationUtils.removeAuthorizationPluginPrivileges(ident, Entity.EntityType.CATALOG);
+    AuthorizationUtils.authorizationPluginRemovePrivileges(ident, Entity.EntityType.CATALOG);
     return dispatcher.dropCatalog(ident, force);
   }
 

diff --git a/core/src/main/java/org/apache/gravitino/hook/FilesetHookDispatcher.java b/core/src/main/java/org/apache/gravitino/hook/FilesetHookDispatcher.java
@@ -94,7 +94,7 @@ public Fileset alterFileset(NameIdentifier ident, FilesetChange... changes)
       }
     }
     if (lastRenameChange != null) {
-      AuthorizationUtils.renameAuthorizationPluginPrivileges(
+      AuthorizationUtils.authorizationPluginRenamePrivileges(
           ident, Entity.EntityType.FILESET, lastRenameChange.getNewName());
     }
 
@@ -104,7 +104,7 @@ public Fileset alterFileset(NameIdentifier ident, FilesetChange... changes)
   @Override
   public boolean dropFileset(NameIdentifier ident) {
     boolean dropped = dispatcher.dropFileset(ident);
-    AuthorizationUtils.removeAuthorizationPluginPrivileges(ident, Entity.EntityType.FILESET);
+    AuthorizationUtils.authorizationPluginRemovePrivileges(ident, Entity.EntityType.FILESET);
     return dropped;
   }
 

diff --git a/core/src/main/java/org/apache/gravitino/hook/SchemaHookDispatcher.java b/core/src/main/java/org/apache/gravitino/hook/SchemaHookDispatcher.java
@@ -90,7 +90,7 @@ public Schema alterSchema(NameIdentifier ident, SchemaChange... changes)
   @Override
   public boolean dropSchema(NameIdentifier ident, boolean cascade) throws NonEmptySchemaException {
     boolean dropped = dispatcher.dropSchema(ident, cascade);
-    AuthorizationUtils.removeAuthorizationPluginPrivileges(ident, Entity.EntityType.SCHEMA);
+    AuthorizationUtils.authorizationPluginRemovePrivileges(ident, Entity.EntityType.SCHEMA);
     return dropped;
   }
 

diff --git a/core/src/main/java/org/apache/gravitino/hook/TableHookDispatcher.java b/core/src/main/java/org/apache/gravitino/hook/TableHookDispatcher.java
@@ -106,7 +106,7 @@ public Table alterTable(NameIdentifier ident, TableChange... changes)
     }
 
     if (lastRenameChange != null) {
-      AuthorizationUtils.renameAuthorizationPluginPrivileges(
+      AuthorizationUtils.authorizationPluginRenamePrivileges(
           ident, Entity.EntityType.TABLE, lastRenameChange.getNewName());
     }
 
@@ -116,14 +116,14 @@ public Table alterTable(NameIdentifier ident, TableChange... changes)
   @Override
   public boolean dropTable(NameIdentifier ident) {
     boolean dropped = dispatcher.dropTable(ident);
-    AuthorizationUtils.removeAuthorizationPluginPrivileges(ident, Entity.EntityType.TABLE);
+    AuthorizationUtils.authorizationPluginRemovePrivileges(ident, Entity.EntityType.TABLE);
     return dropped;
   }
 
   @Override
   public boolean purgeTable(NameIdentifier ident) throws UnsupportedOperationException {
     boolean purged = dispatcher.purgeTable(ident);
-    AuthorizationUtils.removeAuthorizationPluginPrivileges(ident, Entity.EntityType.TABLE);
+    AuthorizationUtils.authorizationPluginRemovePrivileges(ident, Entity.EntityType.TABLE);
     return purged;
   }
 

diff --git a/core/src/main/java/org/apache/gravitino/hook/TopicHookDispatcher.java b/core/src/main/java/org/apache/gravitino/hook/TopicHookDispatcher.java
@@ -89,7 +89,7 @@ public Topic alterTopic(NameIdentifier ident, TopicChange... changes)
   @Override
   public boolean dropTopic(NameIdentifier ident) {
     boolean dropped = dispatcher.dropTopic(ident);
-    AuthorizationUtils.removeAuthorizationPluginPrivileges(ident, Entity.EntityType.TOPIC);
+    AuthorizationUtils.authorizationPluginRemovePrivileges(ident, Entity.EntityType.TOPIC);
     return dropped;
   }