Skip to content

disable optimize_trivial_count in query with array join#17887

Merged
alexey-milovidov merged 2 commits intoClickHouse:masterfrom
sundy-li:trivial_count-fix
Dec 11, 2020
Merged

disable optimize_trivial_count in query with array join#17887
alexey-milovidov merged 2 commits intoClickHouse:masterfrom
sundy-li:trivial_count-fix

Conversation

@sundy-li
Copy link
Copy Markdown
Contributor

@sundy-li sundy-li commented Dec 8, 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):
Trivial query optimization was producing wrong result if query contains ARRAY JOIN (so query is actually non trivial).

Detailed description / Documentation draft:

disable optimize_trivial_count in query with array join

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Dec 8, 2020
@sundy-li
Copy link
Copy Markdown
Contributor Author

sundy-li commented Dec 9, 2020

The failed test is irrelevant.

@alexey-milovidov alexey-milovidov self-assigned this Dec 11, 2020
@alexey-milovidov alexey-milovidov merged commit 0d6fe71 into ClickHouse:master Dec 11, 2020
robot-clickhouse pushed a commit that referenced this pull request Dec 12, 2020
robot-clickhouse pushed a commit that referenced this pull request Dec 12, 2020
alexey-milovidov added a commit that referenced this pull request Dec 12, 2020
Backport #17887 to 20.11: disable optimize_trivial_count in query with array join
alexey-milovidov added a commit that referenced this pull request Dec 12, 2020
Backport #17887 to 20.12: disable optimize_trivial_count in query with array join
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.

3 participants