[BugFix] Fix ranger hive service alter privilege not work (backport #63251) (#63266)

mergify[bot] · Youngwb · web-flow · commit b5cda3b6c4e4 · 2025-09-18T10:18:05.000Z
Co-authored-by: Youngwb &lt;yangwenbo_mailbox@163.com&gt;
diff --git a/fe/fe-core/src/main/java/com/starrocks/authorization/ranger/hive/RangerHiveAccessController.java b/fe/fe-core/src/main/java/com/starrocks/authorization/ranger/hive/RangerHiveAccessController.java
@@ -131,6 +131,8 @@ public String convertToAccessType(PrivilegeType privilegeType) {
             hiveAccessType = HiveAccessType.DROP;
         } else if (privilegeType == PrivilegeType.REFRESH) {
             hiveAccessType = HiveAccessType.REFRESH;
+        } else if (privilegeType == PrivilegeType.ALTER) {
+            hiveAccessType = HiveAccessType.ALTER;
         } else {
             hiveAccessType = HiveAccessType.NONE;
         }
diff --git a/fe/fe-core/src/test/java/com/starrocks/authorization/ranger/RangerInterfaceTest.java b/fe/fe-core/src/test/java/com/starrocks/authorization/ranger/RangerInterfaceTest.java
@@ -270,5 +270,6 @@ public void testHiveConvertToAccessTypeCreate() {
         Assertions.assertEquals("create", controller.convertToAccessType(PrivilegeType.CREATE_DATABASE));
         Assertions.assertEquals("refresh", controller.convertToAccessType(PrivilegeType.REFRESH));
         Assertions.assertEquals("drop", controller.convertToAccessType(PrivilegeType.DROP));
+        Assertions.assertEquals("alter", controller.convertToAccessType(PrivilegeType.ALTER));
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -131,6 +131,8 @@ public String convertToAccessType(PrivilegeType privilegeType) {`
`131`	`131`	`hiveAccessType = HiveAccessType.DROP;`
`132`	`132`	`} else if (privilegeType == PrivilegeType.REFRESH) {`
`133`	`133`	`hiveAccessType = HiveAccessType.REFRESH;`
	`134`	`+ } else if (privilegeType == PrivilegeType.ALTER) {`
	`135`	`+ hiveAccessType = HiveAccessType.ALTER;`
`134`	`136`	`} else {`
`135`	`137`	`hiveAccessType = HiveAccessType.NONE;`
`136`	`138`	`}`
Original file line number	Diff line number	Diff line change
`@@ -270,5 +270,6 @@ public void testHiveConvertToAccessTypeCreate() {`
`270`	`270`	`Assertions.assertEquals("create", controller.convertToAccessType(PrivilegeType.CREATE_DATABASE));`
`271`	`271`	`Assertions.assertEquals("refresh", controller.convertToAccessType(PrivilegeType.REFRESH));`
`272`	`272`	`Assertions.assertEquals("drop", controller.convertToAccessType(PrivilegeType.DROP));`
	`273`	`+ Assertions.assertEquals("alter", controller.convertToAccessType(PrivilegeType.ALTER));`
`273`	`274`	`}`
`274`	`275`	`}`