[SPARK-51795][BUILD] Bump Parquet 1.15.1#50583
[SPARK-51795][BUILD] Bump Parquet 1.15.1#50583CarterFendley wants to merge 3 commits intoapache:branch-3.4from
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