-
Notifications
You must be signed in to change notification settings - Fork 13
Antalya 25.8.14 backport of #87687 - Fix two-level aggregation when using Merge over Distributed #1328
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: antalya-25.8
Are you sure you want to change the base?
Conversation
…merge-distributed Fix two-level aggregation when using Merge over Distributed
…_merge_two_level_aggregation tests: fix 03403_distributed_merge_two_level_aggregation flakiness
tests/queries/0_stateless/03403_distributed_merge_two_level_aggregation.sql
Outdated
Show resolved
Hide resolved
|
The failed stateless tests are also related to Otherwise the actual output of the test shows that the issue has been resolved, which is seen by the line I'll update the |
…uckets_out_of_order_in_aggregation The test failed due to an extra step in actual pipeline output. It is caused by a missing optimization `enable_producing_buckets_out_of_order_in_aggregation` that was added to upstream later and is not worth backporting at this point. The change can be seen at [1]. [1]: https://github.com/ClickHouse/ClickHouse/pull/80179/changes#diff-1806a5c1f13b491c615e887366acf1daac6d85b1528be423e3762705482c80fbR574
arthurpassos
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR also backports ClickHouse#88406 to fix the flaky test added in the original PR
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fixed two-level aggregation when using
MergeoverDistributed(ClickHouse#87687 by @c-end + ClickHouse#88406 by @azat).CI/CD Options
Exclude tests:
Regression jobs to run: