-
Notifications
You must be signed in to change notification settings - Fork 28.9k
[SPARK-49234][BUILD] Upgrade xz to 1.10
#47750
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
|
Since you did |
| python/pyspark/loose_version.py | ||
|
|
||
|
|
||
| BSD 0-Clause |
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 update the binary LICENSE file?
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.
Oh, right.
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.
licenses/LICENSE-xz.txt is added. Thank you.
| @@ -0,0 +1,11 @@ | |||
| Permission to use, copy, modify, and/or distribute this | |||
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.
LuciferYang
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.
LGTM
|
Thank you, @HyukjinKwon and @LuciferYang . |
|
Thank you, @yaooqinn . |
|
Merged into master. Thanks @dongjoon-hyun @HyukjinKwon and @yaooqinn |
|
Thank you again @LuciferYang . |
| @@ -0,0 +1,11 @@ | |||
| Permission to use, copy, modify, and/or distribute this | |||
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.
This file should be placed in the licenses-binary folder as the xz belongs to the binary releases
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.
Thank you, @yaooqinn .
Here is the follow-up.
What changes were proposed in this pull request?
This PR aims to upgrade
xzto1.10independently from Apache Avro change.Why are the changes needed?
1.10is the latest minor version update with new improvements (likeARM64,optimized classes for Java >= 9) and bug fixes.Note that the license is also changed from
Public DomaintoBSD Zero Clause.Does this PR introduce any user-facing change?
No behavior change.
How was this patch tested?
Pass the CIs.
Was this patch authored or co-authored using generative AI tooling?
No.