You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@polaris.apache.org by di...@apache.org on 2026/03/17 14:15:17 UTC

(polaris) branch main updated: Remove unused PolarisAuthorizableOperation values (#3991)

This is an automated email from the ASF dual-hosted git repository.

dimas pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git


The following commit(s) were added to refs/heads/main by this push:
     new 706f507a20 Remove unused PolarisAuthorizableOperation values (#3991)
706f507a20 is described below

commit 706f507a205a862b25eda509d1051787471514c3
Author: Dmitri Bourlatchkov <dm...@gmail.com>
AuthorDate: Tue Mar 17 10:15:04 2026 -0400

    Remove unused PolarisAuthorizableOperation values (#3991)
    
    * Remove unused PolarisAuthorizableOperation values
    
    * `REVOKE_PRINCIPAL_GRANT_FROM_PRINCIPAL_ROLE`
    * `REVOKE_PRINCIPAL_ROLE_GRANT_FROM_PRINCIPAL_ROLE`
---
 CHANGELOG.md                                                          | 1 +
 .../org/apache/polaris/core/auth/PolarisAuthorizableOperation.java    | 4 ----
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 5fa87f5ec3..d07293d170 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -72,6 +72,7 @@ request adding CHANGELOG notes for breaking (!) changes and possibly other secti
 - Changed from Poetry to UV for Python package management.
 - Exclude KMS policies when KMS is not being used for S3.
 - Improved default KMS permission handling to better distinguish read-only and read-write access.
+- Removed unused `PolarisAuthorizableOperation` values: `REVOKE_PRINCIPAL_GRANT_FROM_PRINCIPAL_ROLE`, `REVOKE_PRINCIPAL_ROLE_GRANT_FROM_PRINCIPAL_ROLE`.
 
 ### Deprecations
 
diff --git a/polaris-core/src/main/java/org/apache/polaris/core/auth/PolarisAuthorizableOperation.java b/polaris-core/src/main/java/org/apache/polaris/core/auth/PolarisAuthorizableOperation.java
index 318481709b..86c71b9083 100644
--- a/polaris-core/src/main/java/org/apache/polaris/core/auth/PolarisAuthorizableOperation.java
+++ b/polaris-core/src/main/java/org/apache/polaris/core/auth/PolarisAuthorizableOperation.java
@@ -186,12 +186,8 @@ public enum PolarisAuthorizableOperation {
       SERVICE_MANAGE_ACCESS, PRINCIPAL_ROLE_MANAGE_GRANTS_FOR_GRANTEE),
   LIST_GRANTS_ON_ROOT(SERVICE_MANAGE_ACCESS),
   ADD_PRINCIPAL_GRANT_TO_PRINCIPAL_ROLE(PRINCIPAL_MANAGE_GRANTS_ON_SECURABLE),
-  REVOKE_PRINCIPAL_GRANT_FROM_PRINCIPAL_ROLE(
-      PRINCIPAL_MANAGE_GRANTS_ON_SECURABLE, PRINCIPAL_ROLE_MANAGE_GRANTS_FOR_GRANTEE),
   LIST_GRANTS_ON_PRINCIPAL(PRINCIPAL_LIST_GRANTS),
   ADD_PRINCIPAL_ROLE_GRANT_TO_PRINCIPAL_ROLE(PRINCIPAL_ROLE_MANAGE_GRANTS_ON_SECURABLE),
-  REVOKE_PRINCIPAL_ROLE_GRANT_FROM_PRINCIPAL_ROLE(
-      PRINCIPAL_ROLE_MANAGE_GRANTS_ON_SECURABLE, PRINCIPAL_ROLE_MANAGE_GRANTS_FOR_GRANTEE),
   LIST_GRANTS_ON_PRINCIPAL_ROLE(PRINCIPAL_ROLE_LIST_GRANTS),
   ADD_CATALOG_ROLE_GRANT_TO_CATALOG_ROLE(CATALOG_ROLE_MANAGE_GRANTS_ON_SECURABLE),
   REVOKE_CATALOG_ROLE_GRANT_FROM_CATALOG_ROLE(