Skip to content

Back port #14937 "Fix enable_optimize_predicate_expression for finalizeAggregation" to 20.3 LTS#15231

Merged
alexey-milovidov merged 2 commits intoClickHouse:20.3from
crakjie:backport/20.3/14937
Oct 7, 2020
Merged

Back port #14937 "Fix enable_optimize_predicate_expression for finalizeAggregation" to 20.3 LTS#15231
alexey-milovidov merged 2 commits intoClickHouse:20.3from
crakjie:backport/20.3/14937

Conversation

@crakjie
Copy link
Copy Markdown

@crakjie crakjie commented Sep 24, 2020

Back port #14937 to 20.3 LTS

finalizeAggregation was wrongly marked as stateful, preventing pushing the conditions down.

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)

finalizeAggregation was wrongly marked as stateful, preventing pushing the conditions down.
@robot-clickhouse robot-clickhouse added doc-alert pr-feature Pull request with new product feature labels Sep 24, 2020
@crakjie crakjie changed the title Fix enable_optimize_predicate_expression for finalizeAggregation Back port #14937 "Fix enable_optimize_predicate_expression for finalizeAggregation" to 20.3 LTS Sep 24, 2020
@robot-clickhouse robot-clickhouse added pr-not-for-changelog This PR should not be mentioned in the changelog and removed pr-feature Pull request with new product feature labels Sep 24, 2020
@filimonov filimonov added pr-backport Changes, backported to release branch. Do not use manually - automated use only! and removed doc-alert labels Sep 24, 2020
@alexey-milovidov alexey-milovidov merged commit 556d821 into ClickHouse:20.3 Oct 7, 2020
@alexey-milovidov
Copy link
Copy Markdown
Member

There were issues with "unbundled", "split" and "freebsd" builds, all of them are unrelated to the fix and don't matter for release.
Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backport Changes, backported to release branch. Do not use manually - automated use only! 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.

4 participants