-
Notifications
You must be signed in to change notification settings - Fork 1.6k
docs: Remove disclaimer that datafusion
50.0.0 is not released
#17695
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
base: main
Are you sure you want to change the base?
Conversation
**Note:** DataFusion `50.0.0` has not been released yet. The information provided in this section pertains to features and changes that have already been merged to the main branch and are awaiting release in this version. | ||
You can see the current [status of the `50.0.0 `release here](https://github.com/apache/datafusion/issues/16799) |
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.
Do we need to add a 51.0.0
section now but with this message updated? Or we only add 51.0.0
section when there is something to add?
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.
Good question, I'm not quite sure, since I think 51.0.0
would be empty for now. cc: @alamb
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.
Based off this I think we add the 51.0.0
section: 8d74be8
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.
I agree it would be best to just rename the 50 section to 51
We could also add the appropriate 51 section when we add something that needs an entry in the upgrade guide too as a follow on PR
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.
I've added a section about 51.0.0
.
**Note:** DataFusion `50.0.0` has not been released yet. The information provided in this section pertains to features and changes that have already been merged to the main branch and are awaiting release in this version. | ||
You can see the current [status of the `50.0.0 `release here](https://github.com/apache/datafusion/issues/16799) |
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.
I agree it would be best to just rename the 50 section to 51
We could also add the appropriate 51 section when we add something that needs an entry in the upgrade guide too as a follow on PR
Thanks @nuno-faria and @Jefffrey |
Which issue does this PR close?
Rationale for this change
datafusion
50.0.0 is now released, so we can remove the disclaimer in the upgrading guide.What changes are included in this PR?
upgrading.md
.Are these changes tested?
Docs only.
Are there any user-facing changes?
Docs only.