Skip to content

HIVE-29466: Ambiguous table alias exception when using CTE suggester#6323

Merged
zabetak merged 1 commit intoapache:masterfrom
zabetak:HIVE-29466
Feb 19, 2026
Merged

HIVE-29466: Ambiguous table alias exception when using CTE suggester#6323
zabetak merged 1 commit intoapache:masterfrom
zabetak:HIVE-29466

Conversation

@zabetak
Copy link
Member

@zabetak zabetak commented Feb 17, 2026

What changes were proposed in this pull request?

Consider the alias from the TableSpool operator in the self-join ambiguity check

Why are the changes needed?

Avoid generating an ambiguous SQL and getting a SemanticException when using the CTE suggester which introduces TableSpool operators in the plan.

Does this PR introduce any user-facing change?

No

How was this patch tested?

mvn test -Dtest=TestMiniLlapLocalCliDriver -Dqfile_regex=cbo_self_join.*
mvn test -Dtest=TestMiniLlapLocalCliDriver -Dqfile_regex=cte_.*

@sonarqubecloud
Copy link

Copy link
Contributor

@kasakrisz kasakrisz left a comment

Choose a reason for hiding this comment

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

LGTM

@zabetak zabetak merged commit 54d8741 into apache:master Feb 19, 2026
4 checks passed
@zabetak zabetak deleted the HIVE-29466 branch February 19, 2026 09:03
@zabetak
Copy link
Member Author

zabetak commented Feb 19, 2026

Many thanks for the review @kasakrisz !

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.

3 participants

Comments