Skip to content

Comments

[SPARK-55626][SQL][4.1] Don't load metadata columns on Table unless needed in V2TableUtil#54433

Open
aokolnychyi wants to merge 1 commit intoapache:branch-4.1from
aokolnychyi:spark-55626-4.1
Open

[SPARK-55626][SQL][4.1] Don't load metadata columns on Table unless needed in V2TableUtil#54433
aokolnychyi wants to merge 1 commit intoapache:branch-4.1from
aokolnychyi:spark-55626-4.1

Conversation

@aokolnychyi
Copy link
Contributor

What changes were proposed in this pull request?

This PR prevents loading metadata columns on Table unless needed in V2TableUtil.

Why are the changes needed?

These changed are needed to prevent unnecessary load of metadata columns. In some cases, accessing the metadata columns can lead to exceptions on the connector side as detected in Iceberg. Spark should only ask for metadata columns if they were projected.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

This patch comes with tests.

Was this patch authored or co-authored using generative AI tooling?

No.

… in V2TableUtil

### What changes were proposed in this pull request?

This PR prevents loading metadata columns on Table unless needed in `V2TableUtil`.

### Why are the changes needed?

These changed are needed to prevent unnecessary load of metadata columns. In some cases, accessing the metadata columns can lead to exceptions on the connector side as detected in Iceberg. Spark should only ask for metadata columns if they were projected.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

This patch comes with tests.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes apache#54416 from aokolnychyi/spark-55626.

Authored-by: Anton Okolnychyi <aokolnychyi@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
@aokolnychyi
Copy link
Contributor Author

@dongjoon-hyun, here is a backport of #54416 to 4.1

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM (Pending CIs). Thank you, @aokolnychyi !

@dongjoon-hyun dongjoon-hyun changed the title [SPARK-55626][SQL] Don't load metadata columns on Table unless needed in V2TableUtil [SPARK-55626][SQL][4.1] Don't load metadata columns on Table unless needed in V2TableUtil Feb 24, 2026
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