flyrain commented on code in PR #201:
URL: https://github.com/apache/polaris/pull/201#discussion_r1736832543
##########
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:
We got three yaml files in the directory
1. polaris-management-service.yml
2. rest-catalog-open-api.yaml
3. index.yaml
The index.yaml is actually a generated file combined by 1 and 2. I think
that's where the confusion came from. I was confused as well. Should we remove
the generated file in the repo?
```
docker run -p 8080:80 -v ${PWD}:/spec --rm docker.io/redocly/cli join
spec/docs.yaml spec/polaris-management-service.yml
spec/rest-catalog-open-api.yaml -o spec/index.yaml
--prefix-components-with-info-prop title
```
--
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]
