Skip to content

Fix versioning of aggregate functions (fixes performance tests)#32236

Merged
kssenii merged 2 commits intoClickHouse:masterfrom
azat:fix-agg-func-versioning
Dec 6, 2021
Merged

Fix versioning of aggregate functions (fixes performance tests)#32236
kssenii merged 2 commits intoClickHouse:masterfrom
azat:fix-agg-func-versioning

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Dec 5, 2021

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fix versioning of aggregate functions (fixes performance tests)

Fixes: #24820 (cc @kssenii )

NOTE: marked as Not for changelog since original PR had not been included into any stable release yet

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Dec 5, 2021
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also may be in line 79.. Passing version to set from here

(there is version field in class too)
As I see from a quick look it is the only place where set is used.

@kssenii kssenii self-assigned this Dec 5, 2021
@azat azat force-pushed the fix-agg-func-versioning branch from 91a68e8 to 2b1789a Compare December 5, 2021 18:45
@kssenii kssenii merged commit 8b1bced into ClickHouse:master Dec 6, 2021
@azat azat deleted the fix-agg-func-versioning branch December 6, 2021 06:04
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.

3 participants