flyrain commented on code in PR #906: URL: https://github.com/apache/polaris/pull/906#discussion_r1940291490
########## 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. Review Comment: Is re-generation manual or part of build process? I think we could make it part of build process if it's manual. Not a blocker for this PR though. -- 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]
