-
Notifications
You must be signed in to change notification settings - Fork 29k
[SPARK-51795][BUILD] Bump Parquet 1.15.1 #50583
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
|
Tagging @pan3793 @LuciferYang @cloud-fan @yaooqinn @the-sakthi @dongjoon-hyun for visibility. |
|
branch-3.4 is EOL |
|
Ya, as mentioned in the above, according to Apache Spark versioning policy, no more 3.4.x releases should be expected after that point, even for bug fixes. Let me close this PR to prevent accidental merging. We can continue to discuss on this closed PR, @CarterFendley . |
|
@dongjoon-hyun @HyukjinKwon Yes I understand this. Although I would recommend an exception be made in this case. This is not a bug fix, this is a security patch. The issue in question CVE-2025-30065 currently has the maximum severity vulnerability rating. Spark 3.4 is only a little over a year old, and is likely to have many active users still. |
|
@dongjoon-hyun May I get a comment here? |
|
Especially with a currently max 10/10 CVSS-B score, I think this vulnerability may warrant some exceptions. |
|
I'd recommend to use the latest supported versions when Parquet is ready. AFAIK, there is no complete fix (or ETA) yet, isn't it, @CarterFendley ? |
|
@CarterFendley The CVE only affects |
|
Sorry for the late reply @HyukjinKwon, again I really appreciate your responsiveness on this.
Yep, I am unsure if Can you assure me that these other packages do not use the vulnerable parts of the |
|
As the proposal author of this change, would you mind investigating how it affects Apache Spark, and sharing it since you are asking the exception to make a release in EOL branch? |
|
@HyukjinKwon I can try to take a look, it may be that those modules are unconnected. Although with a CVE of this severity, I would feel better if someone is able to double check me. |
|
I already roughly checked actually, and it doesn't actually affect Apache Spark. But I am asking this to double check :-). |
|
Okay, I think I agree. So looks like the only other module from parquet-java which places a dependency on There is a testing dependency on The only suggestion I have would be to update this spark example which may lead users to install vulnerable versions of Thank you maintainers, appreciate the feedback here ❤️ |
What changes were proposed in this pull request?
Bump Parquet 1.15.1. Backporting #50319
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