Skip to content

ci: split iceberg_spark_test workflow per Iceberg version#4414

Merged
mbutrovich merged 1 commit into
apache:mainfrom
andygrove:ci/split-iceberg-workflow
May 26, 2026
Merged

ci: split iceberg_spark_test workflow per Iceberg version#4414
mbutrovich merged 1 commit into
apache:mainfrom
andygrove:ci/split-iceberg-workflow

Conversation

@andygrove
Copy link
Copy Markdown
Member

Summary

Splits .github/workflows/iceberg_spark_test.yml into one workflow per Iceberg version, mirroring the recent spark_sql_test split (#4408).

  • New iceberg_spark_test_reusable.yml holds the build-native job and the three Iceberg test jobs (iceberg-spark, iceberg-spark-extensions, iceberg-spark-runtime), parameterized by Iceberg / Spark / Java version.
  • Three thin callers — iceberg_spark_test_1_8.yml, iceberg_spark_test_1_9.yml, iceberg_spark_test_1_10.yml — each pin a single Spark version:
    • Iceberg 1.8.1 → Spark 3.4.3 / JDK 11
    • Iceberg 1.9.1 → Spark 3.5.8 / JDK 17
    • Iceberg 1.10.0 → Spark 3.5.8 / JDK 17
  • Only the latest (1.10) triggers on pull requests; 1.8 and 1.9 run on pushes to main only. All three remain workflow_dispatch-able.
  • Updates docs/source/contributor-guide/iceberg-spark-tests.md to describe the new layout.

Test plan

  • actionlint passes on all four new workflow files.
  • Wait for CI to confirm the 1.10 caller triggers on this PR (1.8 / 1.9 should not).
  • After merge, verify push-to-main runs all three callers.

Move job logic into iceberg_spark_test_reusable.yml and add three thin
caller workflows (1.8, 1.9, 1.10). Each caller runs against a single
Spark version: Iceberg 1.8 with Spark 3.4 / JDK 11; Iceberg 1.9 and 1.10
with Spark 3.5 / JDK 17. Only the latest (1.10) runs on pull requests;
1.8 and 1.9 run on pushes to main only.
@andygrove andygrove requested a review from mbutrovich May 23, 2026 13:37
Comment on lines +71 to +77
uses: ./.github/workflows/iceberg_spark_test_reusable.yml
with:
iceberg-short: '1.10'
iceberg-full: '1.10.0'
spark-short: '3.5'
spark-full: '3.5.8'
java: 17
Comment on lines +51 to +57
uses: ./.github/workflows/iceberg_spark_test_reusable.yml
with:
iceberg-short: '1.8'
iceberg-full: '1.8.1'
spark-short: '3.4'
spark-full: '3.4.3'
java: 11
Comment on lines +51 to +57
uses: ./.github/workflows/iceberg_spark_test_reusable.yml
with:
iceberg-short: '1.9'
iceberg-full: '1.9.1'
spark-short: '3.5'
spark-full: '3.5.8'
java: 17
Copy link
Copy Markdown
Contributor

@mbutrovich mbutrovich left a comment

Choose a reason for hiding this comment

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

Thanks @andygrove!

@mbutrovich mbutrovich merged commit a02c815 into apache:main May 26, 2026
11 checks passed
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