Use name instead of id

apache · Jul 11, 2024 · 58762ec · 58762ec
1 parent eb73cad
commit 58762ec
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/core/src/main/java/com/datastrato/gravitino/authorization/PermissionManager.java b/core/src/main/java/com/datastrato/gravitino/authorization/PermissionManager.java
@@ -77,7 +77,7 @@ User grantRolesToUser(String metalake, List<String> roles, String user) {
             List<Long> roleIds = Lists.newArrayList(toRoleIds(roleEntities));
 
             for (RoleEntity roleEntityToGrant : roleEntitiesToGrant) {
-              if (roleNames.contains(roleEntityToGrant.name())) {
+              if (roleIds.contains(roleEntityToGrant.id())) {
                 LOG.warn(
                     "Failed to grant, role {} already exists in the user {} of metalake {}",
                     roleEntityToGrant.name(),
@@ -142,7 +142,7 @@ Group grantRolesToGroup(String metalake, List<String> roles, String group) {
             List<Long> roleIds = Lists.newArrayList(toRoleIds(roleEntities));
 
             for (RoleEntity roleEntityToGrant : roleEntitiesToGrant) {
-              if (roleNames.contains(roleEntityToGrant.name())) {
+              if (roleIds.contains(roleEntityToGrant.id())) {
                 LOG.warn(
                     "Failed to grant, role {} already exists in the group {} of metalake {}",
                     roleEntityToGrant.name(),