Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve Normalization of ASTSelectWithUnionQuery #21246

Merged
merged 5 commits into from Mar 2, 2021

Conversation

ucasfl
Copy link
Collaborator

@ucasfl ucasfl commented Feb 26, 2021

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fix crash in EXPLAIN for query with UNION. Fixes #20876, #21170

Move NormalizeSelectWithUnionQuery from InterpreterSelectWithUnionQuery to executeQuery.

This should close #20876, close #21170, and should fulfill #21003, #21027 at the same time(these two prs can be closed).

@robot-clickhouse robot-clickhouse added the pr-improvement Pull request with some product improvements label Feb 26, 2021
@ucasfl
Copy link
Collaborator Author

ucasfl commented Feb 26, 2021

@ucasfl ucasfl mentioned this pull request Feb 27, 2021
@KochetovNicolai KochetovNicolai merged commit 8e6fa40 into ClickHouse:master Mar 2, 2021
@ucasfl
Copy link
Collaborator Author

ucasfl commented Mar 3, 2021

I think this pr should be backport, it fixed some union related problems. @KochetovNicolai @alexey-milovidov

@filimonov filimonov added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Mar 3, 2021
@robot-clickhouse robot-clickhouse added pr-bugfix Pull request with bugfix, not backported by default and removed pr-improvement Pull request with some product improvements labels Mar 3, 2021
KochetovNicolai added a commit that referenced this pull request Mar 4, 2021
Backport #21246 to 21.1: Improve Normalization of ASTSelectWithUnionQuery
KochetovNicolai added a commit that referenced this pull request Mar 4, 2021
Backport #21246 to 21.2: Improve Normalization of ASTSelectWithUnionQuery
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-bugfix Pull request with bugfix, not backported by default pr-must-backport Pull request should be backported intentionally. Use this label with great care!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

panic when A union B union all C EXPLAIN SYNTAX + UNION DISTINCT in subquery crash
4 participants