Skip to content

Conversation

@cloud-fan
Copy link
Contributor

backport #39077 to 3.3

…columns after alias

This is a followup of apache#38862 . apache#38862 only handles plan alias, but expression alias is missed. This PR fixes it.

fix a regression

Yes, certain querys that failed with `UNRESOLVED_COLUMN` before this PR can work now.

new tests

Closes apache#39077 from cloud-fan/bug.

Authored-by: Wenchen Fan <[email protected]>
Signed-off-by: Max Gekk <[email protected]>
@cloud-fan
Copy link
Contributor Author

@MaxGekk

@github-actions github-actions bot added the SQL label Dec 19, 2022
Copy link
Contributor

@LuciferYang LuciferYang 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

@cloud-fan
Copy link
Contributor Author

@cloud-fan
Copy link
Contributor Author

thanks for review, merging to master!

cloud-fan added a commit that referenced this pull request Dec 22, 2022
…dden columns after alias

backport #39077 to 3.3

Closes #39121 from cloud-fan/backport.

Authored-by: Wenchen Fan <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>
@cloud-fan cloud-fan closed this Dec 22, 2022
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.

3 participants