Skip to content

Handle overflow in quantileTDigest#29075

Merged
vdimir merged 4 commits intoClickHouse:masterfrom
vdimir:quantile-overflow
Sep 24, 2021
Merged

Handle overflow in quantileTDigest#29075
vdimir merged 4 commits intoClickHouse:masterfrom
vdimir:quantile-overflow

Conversation

@vdimir
Copy link
Copy Markdown
Member

@vdimir vdimir commented Sep 16, 2021

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)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
...

Detailed description / Documentation draft:
...

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Sep 16, 2021
@alexey-milovidov alexey-milovidov self-assigned this Sep 16, 2021
@vdimir
Copy link
Copy Markdown
Member Author

vdimir commented Sep 24, 2021

Testcase removed in 152942a (#29075) doesn't work because of timezone

I'm going to rebase on master

@vdimir vdimir force-pushed the quantile-overflow branch 2 times, most recently from 1627560 to 152942a Compare September 24, 2021 10:29
@vdimir vdimir merged commit 908ae1d into ClickHouse:master Sep 24, 2021
@vdimir vdimir deleted the quantile-overflow branch September 24, 2021 12:03
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