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

Fix broken aliases during parsing of special operators #36167

Merged

Conversation

kitaisreal
Copy link
Collaborator

@kitaisreal kitaisreal commented Apr 12, 2022

Changelog category (leave one):

  • Bug Fix (user-visible misbehaviour in official stable or prestable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Queries with aliases inside special operators returned parsing error (was broken in 22.1). Example: SELECT substring('test' AS t, 1, 1).

@kitaisreal kitaisreal added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Apr 12, 2022
@KochetovNicolai KochetovNicolai self-assigned this Apr 12, 2022
@robot-ch-test-poll robot-ch-test-poll added the pr-bugfix Pull request with bugfix, not backported by default label Apr 12, 2022
@kitaisreal kitaisreal added the can be tested Allows running workflows for external contributors label Apr 20, 2022
@kitaisreal kitaisreal force-pushed the special-operators-broken-aliases-fix branch from da39418 to 01e09ba Compare April 21, 2022 11:47
@kitaisreal kitaisreal merged commit bac1f12 into ClickHouse:master Apr 21, 2022
robot-clickhouse pushed a commit that referenced this pull request Apr 22, 2022
robot-clickhouse pushed a commit that referenced this pull request Apr 22, 2022
kitaisreal added a commit that referenced this pull request Apr 28, 2022
Backport #36167 to 22.3: Fix broken aliases during parsing of special operators
kitaisreal added a commit that referenced this pull request Apr 28, 2022
Backport #36167 to 22.4: Fix broken aliases during parsing of special operators
kaynewu added a commit to kaynewu/ClickHouse that referenced this pull request Jun 9, 2022
* Backport ClickHouse#36167 to 22.3: Fix broken aliases during parsing of special operators

* Backport git fix for /build directory

* Trigger CI

* Update version to 22.3.5.20

* Backport ClickHouse#36637 to 22.3: Fix merges of wide parts with type `Object`

* Backport ClickHouse#36487 to 22.3: Add passphrase for certificates

* Backport ClickHouse#36707 to 22.3: Fix vertical merges in wide parts

* Update version to 22.3.6.5

* Backport ClickHouse#36910 to 22.3: Fix bug in keeper which could lead to corrupted compressed logs

* Update version to 22.3.7.5

* Backport ClickHouse#36866 to 22.3: Integration tests

* Backport ClickHouse#35803 to 22.3: Fix bug in indexes of not presented columns in -WithNames formats

* Backport ClickHouse#37021 to 22.3: Fixed problem with infs in `quantileTDigest`

* Backport ClickHouse#36463 to 22.3: Ignore DNS errors when checking if dictionary source is local

* Backport ClickHouse#37443 to 22.3: Functions normalize utf8 fix

* Backport ClickHouse#37336 to 22.3: Fix clash of constant strings in aggregate function, prewhere and join

* Backport ClickHouse#37690 to 22.3: Fix segfault with mysql db + show create table + named collections

Co-authored-by: robot-clickhouse <robot-clickhouse@clickhouse.com>
Co-authored-by: Mikhail f. Shiryaev <felixoid@clickhouse.com>
Co-authored-by: Anton Popov <ap@clickhouse.com>
Co-authored-by: Maksim Kita <maksim@clickhouse.com>
Co-authored-by: Alexey Milovidov <milovidov@clickhouse.com>
Co-authored-by: alesapin <alesapin@clickhouse.com>
Co-authored-by: Kruglov Pavel <48961922+Avogar@users.noreply.github.com>
Co-authored-by: Alexander Tokmakov <tavplubix@clickhouse.com>
Co-authored-by: Vladimir C <vdimir@clickhouse.com>
Co-authored-by: Kseniia Sumarokova <54203879+kssenii@users.noreply.github.com>
@Felixoid Felixoid added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
can be tested Allows running workflows for external contributors pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore 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.

None yet

4 participants