Skip to content

[VL] Spark 4.2: align profile dependency versions (Hadoop 3.5.0, Jackson 2.21.2, Guava, log4j, commons-lang3) - #13161

Merged
baibaichen merged 1 commit into
apache:mainfrom
akshaytayal:spark42-dep-upgrades-main
Sep 30, 2026
Merged

baibaichen merged 1 commit into
apache:mainfrom
akshaytayal:spark42-dep-upgrades-main

Conversation

@akshaytayal

@akshaytayal akshaytayal commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Aligns every Spark-provided library the Gluten spark-4.2 build resolves with the versions
Spark 4.2.0 ships, and fixes one compile break the Hadoop bump introduces.

Spark 4.2 Lib upgrade release notes: https://spark.apache.org/releases/spark-release-4-2-0.html

Versions were taken from apache/spark v4.2.0 pom.xml (the release tag)
and cross-checked with mvn help:evaluate -Pspark-4.2,scala-2.13, so the
effective build versions match — not just what a property literal says.

Dependency Before After Spark 4.2.0
Scala 2.13.17 2.13.18 2.13.18
Hadoop 3.4.1 3.5.0 3.5.0
Jackson (core/databind) 2.18.2 2.21.2 2.21.2
slf4j 2.0.16 2.0.17 2.0.17
Guava 33.4.0-jre 33.6.0-jre 33.6.0-jre
log4j 2.24.3 2.25.4 2.25.4
commons-lang3 3.17.0 3.20.0 3.20.0
antlr4 4.13.1 4.13.1 (unchanged) 4.13.1
arrow 19.0.0 19.0.0 (unchanged) 19.0.0

Scoping notes:

  • Scala is set in the shared scala-2.13 profile (2.13.17). It is overridden
    to 2.13.18 inside the spark-4.2 profile, so only Spark 4.2 builds pick
    it up — Spark 3.4/3.5/4.0/4.1 keep 2.13.17 (verified via help:evaluate).
  • jackson-annotations has no 2.21.2 patch release (published only at the
    minor level, 2.21), so it is pinned separately to 2.21 via a new
    fasterxml.annotations.version property that defaults to ${fasterxml.version}
    at the top level — a no-op for other profiles.
  • Delta / Iceberg / Hudi are external integrations (not Spark-bundled, not in
    Spark's release notes), so they are intentionally left unchanged.

VeloxSparkPlanExecApi.scala — jakarta-safe spill path: Hadoop 3.5.0 drops
javax.ws.rs from the compile classpath, so the spill-path URI rewrite switches
from javax.ws.rs.core.UriBuilder to java.net.URI (JDK-native, same encoded
result).

Why are the changes needed?

The Spark 4.2 profile added in #13126 reused the 4.1 dependency versions. Spark
4.2.0 upgrades these libraries; aligning the profile keeps Gluten's Spark 4.2
build consistent with the Spark runtime it targets and avoids classpath/version
skew.

How was this patch tested?

Full CI on a fork (spark-test 3.4/3.5/4.0/4.1 + scala-2.13 + slow/extended,
tpc-test ubuntu/centos). Only the spark-4.2 profile is changed (plus the
annotations property default, a no-op for other profiles); effective versions
for other Spark builds are unchanged (confirmed with mvn help:evaluate).

Copilot AI balanced review requested due to automatic review settings September 29, 2026 15:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@akshaytayal
akshaytayal force-pushed the spark42-dep-upgrades-main branch from 6c64a73 to 88da8b4 Compare September 29, 2026 15:21
Copilot AI review requested due to automatic review settings September 29, 2026 15:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Align every Spark-provided library the Gluten spark-4.2 build resolves with the
versions Spark 4.2.0 ships (verified against apache/spark v4.2.0 pom.xml and via
mvn help:evaluate -Pspark-4.2,scala-2.13), plus fix the compile break the Hadoop
bump introduces.

Effective versions for a spark-4.2 build, now all matching Spark 4.2.0:
- Scala         2.13.17 -> 2.13.18 (scoped to spark-4.2 profile; overrides the
                shared scala-2.13 profile only for 4.2, other Spark versions keep 2.13.17)
- Hadoop        3.4.1  -> 3.5.0
- Jackson       2.18.2 -> 2.21.2 (annotations pinned to 2.21; no 2.21.2 patch exists)
- slf4j         2.0.16 -> 2.0.17
- Guava         33.4.0-jre -> 33.6.0-jre
- log4j         2.24.3 -> 2.25.4
- commons-lang3 3.17.0 -> 3.20.0
- antlr4 4.13.1, arrow 19.0.0: already matched, unchanged.

jackson-annotations has no 2.21.2 patch release, so it is pinned separately to
2.21 via a new fasterxml.annotations.version property (defaults to
${fasterxml.version} globally -> no-op for other profiles).

VeloxSparkPlanExecApi.scala: Hadoop 3.5.0 drops javax.ws.rs from the compile
classpath, so the spill-path URI rewrite switches from javax.ws.rs.core.UriBuilder
to java.net.URI (JDK-native, same encoded result).

Delta/Iceberg/Hudi are external integrations (not Spark-bundled and not in Spark's
release), so they are intentionally left unchanged.
@akshaytayal
akshaytayal force-pushed the spark42-dep-upgrades-main branch from 88da8b4 to de279fc Compare September 29, 2026 15:36
Copilot AI review requested due to automatic review settings September 29, 2026 15:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions github-actions Bot added CORE works for Gluten Core VELOX labels Sep 29, 2026
@github-actions

Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

2 similar comments
@github-actions

Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions

Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@baibaichen baibaichen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@baibaichen
baibaichen merged commit 4cda2fb into apache:main Sep 30, 2026
63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CORE works for Gluten Core VELOX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants