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 file 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 why the confusion came from. 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]

Reply via email to