-
Notifications
You must be signed in to change notification settings - Fork 176
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Site's REST Catalog Spec Reference to Bundled Polaris + Iceberg API YAML #935
Update Site's REST Catalog Spec Reference to Bundled Polaris + Iceberg API YAML #935
Conversation
weight: 900 | ||
params: | ||
show_page_toc: false | ||
--- | ||
|
||
{{< redoc-polaris "rest-catalog-open-api.yaml" >}} | ||
{{< redoc-polaris "generated/bundled-polaris-catalog-service.yaml" >}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Thanks @HonahX for the fix, and @jackye1995 for the review. |
Hi @flyrain Could you please help me revert this one? It works locally but breaks the published website:
So both management api and catalog api page fails to be rendered :( |
…+ Iceberg API YAML (apache#935)" This reverts commit 3685065.
Hi @jbonofre , the reverting doesn't work as well. Is there any change from Apache Infra so that the website put more security limit?
|
…+ Iceberg API YAML (apache#935)" (apache#942) This reverts commit 3685065.
This PR is the first follow-up of #906
Updated the website's reference to
generated/bundled-polaris-catalog-service.yaml
instead ofrest-catalog-open-api.yaml
. This change ensures that Polaris-native APIs and models can be removed fromrest-catalog-open-api.yaml
in the next step, allowing it to align with the released version.cc: @flyrain @jackye1995