search

tvondra / tdigest

PostgreSQL extension for estimating percentiles using t-digest
PostgreSQL License
87 stars 13 forks source link

force to add centroid if count is big #16

Closed min-mwei closed 3 years ago

min-mwei commented 3 years ago

This is related to https://github.com/tvondra/tdigest/pull/3.

tvondra commented 3 years ago

I pushed a different implementation into the value-count-api branch (a bit more details in pull request #3). I think it's better than just adding a single huge centroid - maybe a bit slower (but not much) and the produced t-digest should be well-formed.

min-mwei commented 3 years ago

Agreed, once you merged the other change, let's close this pull request. Thanks.

min-mwei commented 3 years ago

Superseded by #3