Skip to content

chore: drop Spark 3.4 support - #5358

Open
manuzhang wants to merge 2 commits into
apache:mainfrom
manuzhang:agent/remove-spark-3-4
Open

chore: drop Spark 3.4 support#5358
manuzhang wants to merge 2 commits into
apache:mainfrom
manuzhang:agent/remove-spark-3-4

Conversation

@manuzhang

Copy link
Copy Markdown
Member

Which issue does this PR close?

Closes #4329.

Rationale for this change

Spark 3.4 was deprecated in Comet 1.0.0 and is scheduled for removal in 1.1.0. Dropping it reduces the supported build/test matrix and removes compatibility paths that are no longer needed now that Spark 3.5 is the oldest supported release.

What changes are included in this PR?

  • Remove the Spark 3.4 Maven profile, source/test shims, Spark SQL patch, CI jobs, release artifacts, generated compatibility page, and documentation references.
  • Simplify shared code and tests to assume Spark 3.5 or newer, including removing the Spark 3.4 AQE dynamic partition-pruning fallback.
  • Promote the Spark 3.5 plan-stability overrides into the base golden directories and keep Spark 4.x fallback chains intact.
  • Retarget the Iceberg 1.8 compatibility job to Spark 3.5.9 and JDK 17.

How are these changes tested?

  • make format PROFILES=-Pspark-3.5
  • ./mvnw clean test-compile -DskipTests -Pspark-4.0
  • ./mvnw clean test-compile -DskipTests -Pspark-4.1
  • ./mvnw clean test-compile -DskipTests -Pspark-4.2
  • Focused Spark 3.5 runtime coverage for AQE DPP and hash joins
  • CometTPCDSV1_4_PlanStabilitySuite on Spark 3.5 (97 tests)
  • CometTPCDSV2_7_PlanStabilitySuite on Spark 3.5 (32 tests)
  • YAML, shell syntax, Python bytecode, active-reference, and git diff --check validation

manuzhang and others added 2 commits August 14, 2026 14:58
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
@manuzhang
manuzhang marked this pull request as ready for review August 14, 2026 08:14
value: |
org.apache.comet.exec.CometAggregateSuite
org.apache.comet.exec.CometExec3_4PlusSuite
org.apache.comet.exec.CometExecCompatibilitySuite

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.

Is this used?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, CometExecCompatibilitySuite is renamed from CometExec3_4PlusSuite and used by all supported Spark versions.

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.

Drop support for Spark 3.4

2 participants