dimas-b commented on code in PR #201:
URL: https://github.com/apache/polaris/pull/201#discussion_r1731640502
##########
spec/index.yaml:
##########
@@ -1209,11 +1209,7 @@ paths:
- Polaris_Management_Service_OAuth2: []
delete:
operationId: deleteCatalog
- description: >-
- Delete an existing catalog. This is a cascading operation that deletes
- all metadata, including principals, roles and grants. If the catalog is
- an internal catalog, all tables and namespaces are dropped without
- purge.
+ description: Delete an existing catalog. The catalog must be empty.
Review Comment:
This PR itself LGTM, but I wonder how this spec file is related to the
Iceberg REST Catalog spec... I see some paths match the Iceberg REST paths...
I'd assume this whole file is meant to be 1:1 with the Iceberg spec... WDYT?
--
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]
