Skip to content
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

[Improvement] use incubating version in docs#5114 #5183

Closed
wants to merge 14 commits into from
Closed

[Improvement] use incubating version in docs#5114 #5183

wants to merge 14 commits into from

Conversation

lsyulong
Copy link
Contributor

What changes were proposed in this pull request?

[Improvement] use incubating version in docs#5114

Why are the changes needed?

Gravitino has become an Apache Incubator project since version 0.6.0, so it is necessary to follow the Apache versioning convention like 0.6.0-incubating instead of 0.6.0.

Fix: #5114

Does this PR introduce any user-facing change?

No

How was this patch tested?

No need

* @param headers The base header for Gravitino API.
* @throws NoSuchMetalakeException if the metalake with specified name does not exist.
*/
private GravitinoClient(
Copy link
Contributor

Choose a reason for hiding this comment

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

Are these expected changes in this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, anything other than the document can be ignored

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do I need to close PR, fork the repository again, make modifications based on this repository, and submit PR again

Copy link
Contributor

Choose a reason for hiding this comment

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

@lsyulong
You can remove this unrelated code directly and do not need to close this PR.

@lsyulong lsyulong closed this Oct 21, 2024
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.

[Improvement] use incubating version in docs
2 participants