collado-mike commented on code in PR #201:
URL: https://github.com/apache/polaris/pull/201#discussion_r1733644281
##########
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 spec file is specific to the Polaris management APIs - they aren't part
of the Iceberg catalog spec, nor should they be. There shouldn't be any
correspondence to catalog APIs. The actual Iceberg catalog rest spec that we
implement lives here:
https://github.com/apache/polaris/blob/main/spec/rest-catalog-open-api.yaml
--
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]
