dimas-b commented on code in PR #1353:
URL: https://github.com/apache/polaris/pull/1353#discussion_r2047887344


##########
spec/polaris-management-service.yml:
##########
@@ -1089,6 +1089,10 @@ components:
         clientId:
           type: string
           description: The output-only OAuth clientId associated with this 
principal if applicable
+        federated:

Review Comment:
   Thanks for the clarification! The description of the property LGTM. However, 
I'm not not sure about exposing federated Principals in the Polaris API.
   
   Granted, we have to expose federated Principal Roles in order to allow 
assignment to Catalog Roles, but do Principals have to be exposed?



-- 
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: issues-unsubscr...@polaris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to