-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[branch-51] Update version to 51.0.0, add Changelog #18551
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
Conversation
mbutrovich
left a comment
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.
Looks right to me. Thanks @alamb!
|
Thanks @mbutrovich for the review! |
|
|
||
| # Apache DataFusion 51.0.0 Changelog | ||
|
|
||
| This release consists of 531 commits from 128 contributors. See credits at the end of this changelog for more information. |
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.
128 contributors 😮 -- a new record I think!
## Which issue does this PR close? <!-- We generally require a GitHub issue to be filed for all bug fixes and enhancements and this helps us generate change logs for our releases. You can link an issue to this PR using the GitHub syntax. For example `Closes apache#123` indicates that this PR will close issue apache#123. --> - part of apache#17558 ## Rationale for this change Let's update the version numbers! ## What changes are included in this PR? Update numbers and add changelog. I plan to merge this PR now and I will make a follow on PR (and forward port it to main) with final touchups ## Are these changes tested? <!-- We typically require tests for all PRs in order to: 1. Prevent the code from being accidentally broken by subsequent changes 2. Serve as another way to document the expected behavior of the code If tests are not included in your PR, please explain why (for example, are they covered by existing tests)? --> ## Are there any user-facing changes? <!-- If there are user-facing changes then we may require documentation to be updated before approving the PR. --> <!-- If there are any breaking changes to public APIs, please add the `api change` label. -->
|
Thanks for the review @mbutrovich |
|
I also cherry-picked this to main as well |
## Which issue does this PR close? - part of #17558 - port of #18551 ## Rationale for this change Let's update the version numbers! ## What changes are included in this PR? - forward port the change from #18551 to main ## Are these changes tested? by CI ## Are there any user-facing changes? New version
Which issue does this PR close?
51.0.0(Nov 2025) #17558Rationale for this change
Let's update the version numbers!
What changes are included in this PR?
Update numbers and add changelog.
I plan to merge this PR now and I will make a follow on PR (and forward port it to main) with final touchups
Are these changes tested?
Are there any user-facing changes?