-
Notifications
You must be signed in to change notification settings - Fork 29k
[SPARK-51549][BUILD] Bump Parquet 1.15.1 #50319
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
|
Compared to 1.15.0, only a few lines changed in this patch version, might be good to go branch-4.0? cc @cloud-fan @LuciferYang |
|
The new version seems to include two bug fixes:
and I think they can be backported to branch-4.0. However, I respect @cloud-fan's opinion more. |
|
LGTM! |
dongjoon-hyun
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.
+1, LGTM.
|
Actually, this is a kind of must-have due to the following. Let me merge this. |
### What changes were proposed in this pull request? Bump Parquet 1.15.1. ### Why are the changes needed? Release Notes https://github.com/apache/parquet-java/releases/tag/apache-parquet-1.15.1 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Pass GHA. ### Was this patch authored or co-authored using generative AI tooling? No Closes #50319 from pan3793/SPARK-51549. Authored-by: Cheng Pan <chengpan@apache.org> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org> (cherry picked from commit bd581ec) Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
|
Thank you all. Merged to master/4.0. |
|
late LGTM, thanks for doing it! |
Bump Parquet 1.15.1. Release Notes https://github.com/apache/parquet-java/releases/tag/apache-parquet-1.15.1 No. Pass GHA. No Closes apache#50319 from pan3793/SPARK-51549. Authored-by: Cheng Pan <chengpan@apache.org> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
### What changes were proposed in this pull request? Bump Parquet 1.15.1. ### Why are the changes needed? Release Notes https://github.com/apache/parquet-java/releases/tag/apache-parquet-1.15.1 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Pass GHA. ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#50319 from pan3793/SPARK-51549. Authored-by: Cheng Pan <chengpan@apache.org> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org> (cherry picked from commit 15f6479) Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
What changes were proposed in this pull request?
Bump Parquet 1.15.1.
Why are the changes needed?
Release Notes https://github.com/apache/parquet-java/releases/tag/apache-parquet-1.15.1
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GHA.
Was this patch authored or co-authored using generative AI tooling?
No