travis-bowen commented on code in PR #2697:
URL: https://github.com/apache/polaris/pull/2697#discussion_r2393364064
##########
polaris-core/src/main/java/org/apache/polaris/core/entity/PolarisPrivilege.java:
##########
@@ -155,6 +155,91 @@ public enum PolarisPrivilege {
PolarisEntityType.POLICY,
PolarisEntitySubType.NULL_SUBTYPE,
PolarisEntityType.CATALOG_ROLE),
+ TABLE_ASSIGN_UUID(
Review Comment:
I'm not sure how much value that might add for users.
My main reasons are - it's hard for me to imagine that someone would grant
everything except adding snapshots to a role. At that point, I'd think someone
would just grant TABLE_WRITE_PROPERTIES which already encompasses everything.
The two snapshot only as a group seem a little more useful to me though it's
also only two at this point.
I'd also hesitate if there's a differing opinions on how to group the
privileges which does get a little trickier around which snapshot ones belong
together.
What do you think?
--
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]
