Skip to content

Fix usage of WITH FILL together with WITH TOTALS#25539

Merged
CurtizJ merged 3 commits intoClickHouse:masterfrom
CurtizJ:with-fill-totals
Jun 23, 2021
Merged

Fix usage of WITH FILL together with WITH TOTALS#25539
CurtizJ merged 3 commits intoClickHouse:masterfrom
CurtizJ:with-fill-totals

Conversation

@CurtizJ
Copy link
Copy Markdown
Member

@CurtizJ CurtizJ commented Jun 21, 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 wrong totals for query WITH TOTALS and WITH FILL. Fixes #20872.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Jun 21, 2021
@KochetovNicolai KochetovNicolai self-assigned this Jun 21, 2021
@CurtizJ CurtizJ merged commit 93f15e4 into ClickHouse:master Jun 23, 2021
robot-clickhouse pushed a commit that referenced this pull request Jun 23, 2021
robot-clickhouse pushed a commit that referenced this pull request Jun 23, 2021
robot-clickhouse pushed a commit that referenced this pull request Jun 23, 2021
alexey-milovidov added a commit that referenced this pull request Jun 23, 2021
Backport #25539 to 21.6: Fix usage of `WITH FILL` together with `WITH TOTALS`
alexey-milovidov added a commit that referenced this pull request Jun 23, 2021
Backport #25539 to 21.5: Fix usage of `WITH FILL` together with `WITH TOTALS`
alexey-milovidov added a commit that referenced this pull request Jun 23, 2021
Backport #25539 to 21.3: Fix usage of `WITH FILL` together with `WITH TOTALS`
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.

WITH FILL FROM TO breaks WITH TOTALS result

3 participants