Skip to content

Fixed problem with infs in quantileTDigest#37021

Merged
Avogar merged 6 commits intoClickHouse:masterfrom
excitoon-favorites:fixtdigest
May 16, 2022
Merged

Fixed problem with infs in quantileTDigest#37021
Avogar merged 6 commits intoClickHouse:masterfrom
excitoon-favorites:fixtdigest

Conversation

@excitoon
Copy link
Copy Markdown
Contributor

@excitoon excitoon commented May 8, 2022

Changelog category (leave one):

  • Bug Fix (user-visible misbehaviour in official stable or prestable release)

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

Fixed problem with infs in quantileTDigest.
Fixes #32107

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-bugfix Pull request with bugfix, not backported by default label May 8, 2022
@Avogar Avogar self-assigned this May 9, 2022
@Avogar
Copy link
Copy Markdown
Member

Avogar commented May 9, 2022

Can we write a test for it?

Copy link
Copy Markdown
Member

@Avogar Avogar left a comment

Choose a reason for hiding this comment

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

@excitoon excitoon requested a review from Avogar May 16, 2022 12:02
@Avogar
Copy link
Copy Markdown
Member

Avogar commented May 16, 2022

LGTM

@Avogar Avogar merged commit 5e34f48 into ClickHouse:master May 16, 2022
@den-crane
Copy link
Copy Markdown
Contributor

Can it be backported into 22.3 LTS ?

Avogar added a commit that referenced this pull request May 20, 2022
Backport #37021 to 21.8: Fixed problem with infs in `quantileTDigest`
Avogar added a commit that referenced this pull request May 20, 2022
Backport #37021 to 22.3: Fixed problem with infs in `quantileTDigest`
Avogar added a commit that referenced this pull request May 23, 2022
Backport #37021 to 22.4: Fixed problem with infs in `quantileTDigest`
kaynewu added a commit to kaynewu/ClickHouse that referenced this pull request Jun 9, 2022
* Backport ClickHouse#36167 to 22.3: Fix broken aliases during parsing of special operators

* Backport git fix for /build directory

* Trigger CI

* Update version to 22.3.5.20

* Backport ClickHouse#36637 to 22.3: Fix merges of wide parts with type `Object`

* Backport ClickHouse#36487 to 22.3: Add passphrase for certificates

* Backport ClickHouse#36707 to 22.3: Fix vertical merges in wide parts

* Update version to 22.3.6.5

* Backport ClickHouse#36910 to 22.3: Fix bug in keeper which could lead to corrupted compressed logs

* Update version to 22.3.7.5

* Backport ClickHouse#36866 to 22.3: Integration tests

* Backport ClickHouse#35803 to 22.3: Fix bug in indexes of not presented columns in -WithNames formats

* Backport ClickHouse#37021 to 22.3: Fixed problem with infs in `quantileTDigest`

* Backport ClickHouse#36463 to 22.3: Ignore DNS errors when checking if dictionary source is local

* Backport ClickHouse#37443 to 22.3: Functions normalize utf8 fix

* Backport ClickHouse#37336 to 22.3: Fix clash of constant strings in aggregate function, prewhere and join

* Backport ClickHouse#37690 to 22.3: Fix segfault with mysql db + show create table + named collections

Co-authored-by: robot-clickhouse <[email protected]>
Co-authored-by: Mikhail f. Shiryaev <[email protected]>
Co-authored-by: Anton Popov <[email protected]>
Co-authored-by: Maksim Kita <[email protected]>
Co-authored-by: Alexey Milovidov <[email protected]>
Co-authored-by: alesapin <[email protected]>
Co-authored-by: Kruglov Pavel <[email protected]>
Co-authored-by: Alexander Tokmakov <[email protected]>
Co-authored-by: Vladimir C <[email protected]>
Co-authored-by: Kseniia Sumarokova <[email protected]>
@Felixoid Felixoid added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-bugfix Pull request with bugfix, not backported by default pr-must-backport Pull request should be backported intentionally. Use this label with great care!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error with AggregateFunction quantilesTDigest causing part merges to fail

6 participants