adutra commented on code in PR #1397:
URL: https://github.com/apache/polaris/pull/1397#discussion_r2050607126


##########
polaris-core/src/main/java/org/apache/polaris/core/auth/AuthenticatedPolarisPrincipal.java:
##########
@@ -48,14 +48,23 @@ public PolarisEntity getPrincipalEntity() {
     return principalEntity;
   }
 
+  /**
+   * Returns the set of activated principal role names. Activated role names 
are the roles that were
+   * explicitly requested by the client when authenticating, through JWT 
claims or other means.
+   *
+   * <p>By convention, this method returns an empty set when the principal is 
requesting all
+   * available principal roles.
+   */

Review Comment:
   I'm trying to document things as I go. This is the best knowledge I can get 
for this principal attribute.



-- 
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]

Reply via email to