Skip to content

Fix 'LIMIT WITH TIES' with aliases#11689

Merged
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
CurtizJ:fix-with-ties
Jun 16, 2020
Merged

Fix 'LIMIT WITH TIES' with aliases#11689
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
CurtizJ:fix-with-ties

Conversation

@CurtizJ
Copy link
Copy Markdown
Member

@CurtizJ CurtizJ commented Jun 15, 2020

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 LIMIT n WITH TIES usage together with ORDER BY statement, which contains aliases.

Fixes #8665.

@blinkov blinkov added the pr-bugfix Pull request with bugfix, not backported by default label Jun 15, 2020
@alexey-milovidov alexey-milovidov self-assigned this Jun 15, 2020
@alexey-milovidov alexey-milovidov merged commit 82b27ab into ClickHouse:master Jun 16, 2020
vitlibar pushed a commit that referenced this pull request Jun 19, 2020
Fix 'LIMIT WITH TIES' with aliases

(cherry picked from commit 82b27ab)
CurtizJ added a commit that referenced this pull request Jul 27, 2020
Backport #11689 to 20.3: Fix 'LIMIT WITH TIES' with aliases
@CurtizJ CurtizJ deleted the fix-with-ties branch July 31, 2020 18:59
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Not found column when 'WITH TIES' is used.

4 participants