HonahX commented on code in PR #1529:
URL: https://github.com/apache/polaris/pull/1529#discussion_r2096225357
##########
polaris-core/src/main/java/org/apache/polaris/core/entity/PolarisPrivilege.java:
##########
@@ -150,6 +150,11 @@ public enum PolarisPrivilege {
CATALOG_DETACH_POLICY(81, PolarisEntityType.CATALOG),
NAMESPACE_DETACH_POLICY(82, PolarisEntityType.NAMESPACE),
TABLE_DETACH_POLICY(83, PolarisEntityType.TABLE_LIKE,
PolarisEntitySubType.ICEBERG_TABLE),
+ POLICY_MANAGE_GRANTS_ON_SECURABLE(
Review Comment:
This along with
- NAMESPACE_MANAGED_GRANTS_ON_SECURABLE
- TABLE_MANAGED_GRANTS_ON_SECURABLE
- VIEW_MANAGED_GRANTS_ON_SECURABLE
- CATALOG_MANAGED_GRANTS_ON_SECURABLE
forms the `CATALOG_MANAGE_ACCESS`. We currently do not allow granting these
sub-privileges through management APIs. I just follow the pattern to ensure
consistency and flexibility to make the privilege granting more fine-grained in
the future if needed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
