Skip to content

Conversation

@pan3793
Copy link
Member

@pan3793 pan3793 commented Mar 19, 2025

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

@github-actions github-actions bot added the BUILD label Mar 19, 2025
@pan3793
Copy link
Member Author

pan3793 commented Mar 19, 2025

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

@LuciferYang
Copy link
Contributor

LuciferYang commented Mar 19, 2025

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.

@the-sakthi
Copy link
Member

LGTM!

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM.

@dongjoon-hyun
Copy link
Member

Actually, this is a kind of must-have due to the following. Let me merge this.

dongjoon-hyun pushed a commit that referenced this pull request Mar 21, 2025
### 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>
@dongjoon-hyun
Copy link
Member

Thank you all. Merged to master/4.0.

@cloud-fan
Copy link
Contributor

late LGTM, thanks for doing it!

wangyum pushed a commit to wangyum/spark that referenced this pull request Apr 7, 2025
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>
zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 14, 2025
### 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants