Skip to content

Publish 1.0.0 documents #1910

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Publish 1.0.0 documents #1910

wants to merge 1 commit into from

Conversation

flyrain
Copy link
Contributor

@flyrain flyrain commented Jun 18, 2025

We published the 0.9 doc after the release(#1175). Similarly, I will make this PR ready to review once 1.0 release was done.
Local test works well:
Screenshot 2025-06-18 at 3 42 40 PM

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR publishes the updated documentation for the Polaris 1.0.0 release, reflecting new features, deployment guides, and configuration details.

  • Added new documentation pages for Polaris Spark client, management service, catalog service, metastores, and various getting-started guides.
  • Introduced deployment guides for AWS, Azure, GCP and updated detailed configuration and production guidelines.

Reviewed Changes

Copilot reviewed 20 out of 28 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
site/content/in-dev/1.0.0/polaris-spark-client.md New documentation for the Polaris Spark client.
site/content/in-dev/1.0.0/polaris-management-service.md Added documentation for the Polaris Management Service.
site/content/in-dev/1.0.0/polaris-catalog-service.md Added documentation for the Catalog API Spec.
site/content/in-dev/1.0.0/metastores.md New documentation detailing metastores configuration and usage.
site/content/in-dev/1.0.0/getting-started/using-polaris.md Guide for using Polaris via CLI and API.
site/content/in-dev/1.0.0/getting-started/quickstart.md Quickstart guide for deploying Polaris locally and via Docker.
site/content/in-dev/1.0.0/getting-started/install-dependencies.md Installation guide covering prerequisites.
site/content/in-dev/1.0.0/getting-started/deploying-polaris/quickstart-deploy-gcp.md Deployment guide for Google Cloud Platform.
site/content/in-dev/1.0.0/getting-started/deploying-polaris/quickstart-deploy-azure.md Deployment guide for Microsoft Azure.
site/content/in-dev/1.0.0/getting-started/deploying-polaris/quickstart-deploy-aws.md Deployment guide for Amazon Web Services with a noted typo.
site/content/in-dev/1.0.0/deploying-polaris/_index.md Index page addressing cloud provider deployments.
site/content/in-dev/1.0.0/getting-started/_index.md Getting started overview page.
site/content/in-dev/1.0.0/generic-table.md Documentation for Generic Table (Beta) functionality.
site/content/in-dev/1.0.0/evolution.md Document describing the Polaris evolution strategy and upgrade considerations.
site/content/in-dev/1.0.0/entities.md Overview of the various entities managed in Polaris.
site/content/in-dev/1.0.0/configuring-polaris-for-production.md Production configuration guidelines and checklist.
site/content/in-dev/1.0.0/configuration.md Detailed configuration reference for Polaris (includes a potential typo).
site/content/in-dev/1.0.0/admin-tool.md Documentation for the Polaris administration tool.
site/content/in-dev/1.0.0/access-control.md Comprehensive guide on access control and RBAC in Polaris.
site/content/in-dev/1.0.0/_index.md Overview page for the 1.0.0 documentation set.

```

## Next Steps
Congrats, you now have a running instance of1 Polaris! For details on how to use Polaris, check out the [Using Polaris]({{% relref "../using-polaris.md" %}}) page.
Copy link
Preview

Copilot AI Jun 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a typo in the message: 'instance of1 Polaris' should be corrected to 'instance of Polaris'.

Suggested change
Congrats, you now have a running instance of1 Polaris! For details on how to use Polaris, check out the [Using Polaris]({{% relref "../using-polaris.md" %}}) page.
Congrats, you now have a running instance of Polaris! For details on how to use Polaris, check out the [Using Polaris]({{% relref "../using-polaris.md" %}}) page.

Copilot uses AI. Check for mistakes.

|----------------------------------------------------------------------------------------|-----------------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
| `polaris.persistence.type` | `relational-jdbc` | Define the persistence backend used by Polaris (`in-memory`, `relational-jdbc`, `eclipse-link` (deprecated)). See [Configuring Apache Polaris for Production)[{{% ref "configuring-polaris-for-production.md" %}}) |
| `polaris.persistence.relational.jdbc.max-retries` | `1` | Total number of retries JDBC persistence will attempt on connection resets or serialization failures before giving up. |
| `polaris.persistence.relational.jdbc.max_duaration_in_ms` | `5000 ms` | Max time interval (ms) since the start of a transaction when retries can be attempted. |
Copy link
Preview

Copilot AI Jun 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The property name 'max_duaration_in_ms' appears to contain a typo. It might be intended to be 'max_duration_in_ms'.

Suggested change
| `polaris.persistence.relational.jdbc.max_duaration_in_ms` | `5000 ms` | Max time interval (ms) since the start of a transaction when retries can be attempted. |
| `polaris.persistence.relational.jdbc.max_duration_in_ms` | `5000 ms` | Max time interval (ms) since the start of a transaction when retries can be attempted. |

Copilot uses AI. Check for mistakes.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

copilot did a good job to find typos! We should fix it in both places.

Copy link
Member

@jbonofre jbonofre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks !

NB: we should try to use versioned-docs branch. See 237ce0c

@github-project-automation github-project-automation bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board Jun 20, 2025
@flyrain
Copy link
Contributor Author

flyrain commented Jun 20, 2025

NB: we should try to use versioned-docs branch. See 237ce0c

Thanks @jbonofre ! What's the plan of using the branch? How does it work with the current site?

@snazy
Copy link
Member

snazy commented Jun 23, 2025

These docs will appear in the URL https://polaris.apache.org/in-dev/1.0.0/ instead of https://polaris.apache.org/releases/1.0.0/. IMHO "in-dev" is not appropriate for a release.

@flyrain
Copy link
Contributor Author

flyrain commented Jun 23, 2025

These docs will appear in the URL https://polaris.apache.org/in-dev/1.0.0/ instead of https://polaris.apache.org/releases/1.0.0/. IMHO "in-dev" is not appropriate for a release.

That makes sense. Considering the 0.9 doc is also in in-dev, how about refactor it in a follow-up PR? Here is my plan, right after this PR. I will move both 0.9 and 1.0.0 to a new dir releases, and we could also combine the in-dev and unreleased to either unreleased.

@snazy
Copy link
Member

snazy commented Jun 23, 2025

Why refactor? Just push the versioned docs to the versioned-docs branch and those will automatically be accessible under .../releases/....

@jbonofre
Copy link
Member

The reason why I pushed in in-dev is because it's 0.9.x (not the release), as an active branch. I'm fine to push only 0.9.0 or release numbers in versioned-docs branch and so in /releases.

@flyrain
Copy link
Contributor Author

flyrain commented Jun 23, 2025

The directory name is 0.9.0 and the doc inside are intended for that particular release. At least, any site visitor will consider so. The reason to refactor it is to move 0.9.0 as well, and follow the previous pattern done in the previous release.
Screenshot 2025-06-23 at 11 21 35 AM

@jbonofre
Copy link
Member

Yes agree. Just explained the initial reason.

@snazy
Copy link
Member

snazy commented Jun 24, 2025

But why not doing it now right away instead of another PR?

@flyrain
Copy link
Contributor Author

flyrain commented Jun 25, 2025

As I said at #1910 (comment). The refactor needs to cover 0.9.0 as well. The current PR is consistent with the previous behavior. I'm actually quite concern that once I made the change you suggest, someone may point out "why are you not follow the convention?". Believe or not, It happens to me a lot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants