Skip to content

fix(spark-runtime): Support Parquet source partitioning (#927) - #931

Merged
vinishjail97 merged 3 commits into
apache:mainfrom
zhang-arvin:fix/927-parquet-partition-spec
Oct 23, 2026
Merged

vinishjail97 merged 3 commits into
apache:mainfrom
zhang-arvin:fix/927-parquet-partition-spec

Conversation

@zhang-arvin

Copy link
Copy Markdown
Contributor

Fixes #927

@the-other-tim-brown

Copy link
Copy Markdown
Contributor

@zhang-arvin can we have this target main instead? if we want to fix this on the 0.4.0 release we can start a thread to raise a 0.4.1 release and then cherry-pick this to the release branch.

@vinishjail97

Copy link
Copy Markdown
Contributor

@zhang-arvin can we have this target main instead? if we want to fix this on the 0.4.0 release we can start a thread to raise a 0.4.1 release and then cherry-pick this to the release branch.

@zhang-arvin I think this needs the xtable-spark-runtime to be merged in main. If you want to take over my existing work feel on xtable-spark-runtime using hudi version 1.2.1, free to do so. I can get to it in few days.
#838
#839

@zhang-arvin

Copy link
Copy Markdown
Contributor Author

Checked the prerequisite before retargeting: xtable-spark-runtime is not on main yet (it only exists on branch-0.4), so this PR can't be rebased onto main as-is — three of the five files it touches (xtable-spark-runtime/.../TableSyncSpec.java, XTableSparkSync.java, XTableSyncService.java) don't exist there.

@vinishjail97 the two runtime PRs you linked — #838 and #839 — are still open and in draft, targeting main, so the spark-runtime isn't in main yet. So I'm leaving the base as branch-0.4 for now: the PR is mergeable and the change itself is small (route the partition spec through ParquetSourceConfig.PARTITION_FIELD_SPEC_CONFIG for Parquet sources instead of the Hudi key).

Plan:

@the-other-tim-brown the 0.4.x cherry-pick path is orthogonal and fine by me — this change is small enough to cherry-pick cleanly once it's on main.

@vinishjail97
vinishjail97 changed the base branch from branch-0.4 to main October 19, 2026 20:12
@vinishjail97 vinishjail97 changed the title [Bug] Support Parquet source partitioning in spark-runtime (#927) fix(spark-runtime): Support Parquet source partitioning (#927) Oct 23, 2026
@vinishjail97
vinishjail97 force-pushed the fix/927-parquet-partition-spec branch from 7da41d9 to e54891c Compare October 23, 2026 00:03
@vinishjail97
vinishjail97 merged commit d34a37a into apache:main Oct 23, 2026
7 of 8 checks passed
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.

spark-runtime: no way to configure Parquet source partitioning; targets always come out unpartitioned

3 participants