Skip to content

[branch-53] correct parquet leaf index mapping when schema contains struct cols (#20698) - #20884

Merged
alamb merged 1 commit into
apache:branch-53from
alamb:alamb/backport_20698_branch53
Mar 12, 2026
Merged

[branch-53] correct parquet leaf index mapping when schema contains struct cols (#20698)#20884
alamb merged 1 commit into
apache:branch-53from
alamb:alamb/backport_20698_branch53

Conversation

@alamb

@alambalamb commented Mar 11, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

@friendlymatthew@alamb
correct parquet leaf index mapping when schema contains struct cols (a…
4505c01
…pache#20698)
## Which issue does this PR close?
- Closesapache#20695
## Rationale for this change
Row filter pushdown assumed Arrow field indices equal Parquet leaf
indices, which breaks when Struct columns are present because their
children expand into separate leaves and shift all subsequent indices
@github-actionsgithub-actionsBot added sqllogictest SQL Logic Tests (.slt) datasource Changes to the datasource crate labels Mar 11, 2026
comphead
comphead approved these changes Mar 11, 2026
View reviewed changes

@compheadcomphead 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.

Thanks @alamb

@alamb
alamb merged commit e922af9 into apache:branch-53Mar 12, 2026
29 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@compheadcompheadcomphead approved these changes

Assignees

No one assigned

Labels

datasourceChanges to the datasource cratesqllogictestSQL Logic Tests (.slt)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alamb@comphead@friendlymatthew