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]

Reply via email to