Skip to content

[AURON #2364] fix: Native Paimon V2 scan returns wrong results with native filter#2365

Merged
SteNicholas merged 2 commits into
apache:masterfrom
lyne7-sc:fix/paimon_rename
Jun 29, 2026
Merged

[AURON #2364] fix: Native Paimon V2 scan returns wrong results with native filter#2365
SteNicholas merged 2 commits into
apache:masterfrom
lyne7-sc:fix/paimon_rename

Conversation

@lyne7-sc

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #2364

Rationale for this change

Auron recently added native scan support for Paimon DSv2 tables. When a Paimon V2 native scan is followed by a native filter, the query can return wrong results.

For example:

create table paimon.db.t_rename_columns (id int, v string) using paimon;
insert into paimon.db.t_rename_columns values (1, 'a'), (2, 'b');
select id from paimon.db.t_rename_columns where id = 1;

The buggy plan returns an empty result instead of [1].

What changes are included in this PR?

updates AuronConverters.needRenameColumns to mark NativePaimonV2TableScan as requiring rename-columns handling.

Are there any user-facing changes?

No API changes. Only a bug fix.

How was this patch tested?

Adds a Paimon V2 integration test covering a native filter on top of a native Paimon V2 scan.

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@SteNicholas SteNicholas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@lyne7-sc, thanks for contribution. LGTM. The fix is mechanistically correct and complete.

@SteNicholas SteNicholas merged commit ebf25a2 into apache:master Jun 29, 2026
123 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Native paimon v2 scan returns wrong results with native filter

3 participants