Skip to content

[SPARK-58869][BUILD] Upgrade Parquet to 1.18.0 - #58120

Open
iemejia wants to merge 1 commit into
apache:masterfrom
iemejia:parquet-dep-update
Open

[SPARK-58869][BUILD] Upgrade Parquet to 1.18.0#58120
iemejia wants to merge 1 commit into
apache:masterfrom
iemejia:parquet-dep-update

Conversation

@iemejia

@iemejia iemejia commented Aug 19, 2026

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

This PR upgrades Apache Parquet from 1.17.1 to 1.18.0. It bumps the
parquet.version property in pom.xml and updates the dependency manifest
dev/deps/spark-deps-hadoop-3-hive-2.3 accordingly.

Why are the changes needed?

To keep the Parquet dependency up to date and pick up the bug fixes and
improvements shipped in the 1.18.0 release.

Release notes: https://github.com/apache/parquet-java/releases/tag/apache-parquet-1.18.0

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Existing tests via CI. This is a dependency version bump with the
dependency manifest updated accordingly.

Was this patch authored or co-authored using generative AI tooling?

No.

@pan3793

pan3793 commented Aug 19, 2026

Copy link
Copy Markdown
Member

@Fokko opened a PR (#57406) for this a month ago, but has had no response or follow-up since then. In addition, I see there is a data corruption report raised in the Parquet dev mailing list https://lists.apache.org/thread/zy4ox06ocjo4c7jm76xddvbgymzcjsmt, given that, maybe we should hold this and wait for Parquet 1.18.1?

@iemejia

iemejia commented Aug 19, 2026

Copy link
Copy Markdown
Member Author

Sorry, I hadn't noticed @Fokko's PR. My version is slightly more comprehensive, but I'm fine with using the other one. The important part is to wait for the 1.18.1 release, which contains the fix for this issue.

@uros-b

uros-b commented Aug 19, 2026

Copy link
Copy Markdown
Member

oh I only now see the comments above, I'm fine with your alignment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants