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?
The design doc does talk about Polaris entities for federated roles, but I
did not see it mention having to define entities for federated principals. Per
section `Automatic User Provisioning` there is no strict requirement for
federated principal entities... hence my question.
--
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]
