collado-mike commented on code in PR #808:
URL: https://github.com/apache/polaris/pull/808#discussion_r1956662293
##########
spec/rest-catalog-open-api.yaml:
##########
@@ -1595,6 +1595,261 @@ paths:
$ref: '#/components/responses/ServiceUnavailableResponse'
5XX:
$ref: '#/components/responses/ServerErrorResponse'
+
+ /v1/{prefix}/namespaces/{namespace}/policies:
Review Comment:
Personally, I would start with policies at the Catalog level, with the
ability to add policy overrides at the namespace level later on. It is more
consistent with how we manage privileges and storage configuration. Both start
at the catalog level and overrides are attached to the child entities. Given
that the mapping still happens at the namespace, I think CRUD at the Catalog
level makes sense.
--
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]