jackye1995 commented on code in PR #906: URL: https://github.com/apache/polaris/pull/906#discussion_r1936116207
########## spec/README.md: ########## @@ -0,0 +1,29 @@ +# Polaris API Specifications +Polaris provides two sets of OpenAPI specifications: +- `polaris-management-service.yml` - Defines the management APIs for using Polaris to create and manage Iceberg catalogs and their principals +- `polaris-catalog-service.yaml` - Defines the specification for the Polaris Catalog API, which encompasses both the Iceberg REST Catalog API + and Polaris-native API. + - `polaris-apis` - Contains the specifications of Polaris-native API + - `rest-catalog-open-api.yaml` - Contains the specification for Iceberg Rest Catalog API + +## Generated Specification Files +The specification files in the generated folder are automatically created using OpenAPI bundling tools such as +[Redocly CLI](https://github.com/Redocly/redocly-cli). + +These files should not be manually edited (except adding license header). They are intended for preview purposes only, +such as rendering a preview on a website. + +Whenever the source specification files are updated, the generated files must be re-generated to reflect those changes. + +Below are steps to generate `bundled-polaris-catalog-service.yaml` Review Comment: do we need to also commit the generated file? If possible I think we should have that in .gitignore -- 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]
