Skip to content

Backport unrelated changes, which fixes aliases bug#25681

Merged
alesapin merged 2 commits into21.3from
fix_strange_bug_2
Jun 25, 2021
Merged

Backport unrelated changes, which fixes aliases bug#25681
alesapin merged 2 commits into21.3from
fix_strange_bug_2

Conversation

@alesapin
Copy link
Copy Markdown
Member

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

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Found these changes using git bisect. Error looks:

2021.06.24 15:44:03.469239 [ 56078 ] {c7731976-cfbc-4430-b458-e169387dc2ba} <Error> executeQuery: Code: 179, e.displayText() = DB::Exception: Different expressions with the same alias x:                         
identity(multiIf(ContextType IN (3, 4, 11), 0, ContextType IN (7, 8), TargetPhraseID, PhraseID)) AS x                                                                                                                                          
and                                                                                                                                                                                                                                            
identity(PhraseProcessedID) AS x                                                                                                                                                                                                               
: While processing identity(multiIf(ContextType IN (3, 4, 11), 0, ContextType IN (7, 8), TargetPhraseID, PhraseID)) AS x (version 21.6.1.1) (from [::1]:48452) (in query: SELECT identity(PhraseProcessedID) AS x FROM distributed_tbl WHERE (D
ate = toDate('2016-03-01')) AND (ClientID = 1486392) AND (x = 0);), Stack trace (when copying this message, always include the lines below):                                                                                                   

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Jun 24, 2021
@alesapin
Copy link
Copy Markdown
Member Author

E OSError: [Errno 28] No space left on device in integration tests

@alesapin
Copy link
Copy Markdown
Member Author

Ok, let's merge and check release branch.

@alesapin alesapin merged commit adf2ff7 into 21.3 Jun 25, 2021
@alesapin alesapin deleted the fix_strange_bug_2 branch June 25, 2021 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants