Skip to content

Remove hppc from percentiles agg#86023

Merged
rjernst merged 2 commits intoelastic:masterfrom
rjernst:hppc/perc_agg
Apr 25, 2022
Merged

Remove hppc from percentiles agg#86023
rjernst merged 2 commits intoelastic:masterfrom
rjernst:hppc/perc_agg

Conversation

@rjernst
Copy link
Copy Markdown
Member

@rjernst rjernst commented Apr 20, 2022

The percentiles bucket agg uses an hppc arraylist of doubles to store
the parsed percent values. This is a very small list and does not need
to be a native array. This commit changes to using a standard ArrayList.

relates #84735

The percentiles bucket agg uses an hppc arraylist of doubles to store
the parsed percent values. This is a very small list and does not need
to be a native array. This commit changes to using a standard ArrayList.

relates elastic#84735
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Apr 20, 2022
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

@rjernst rjernst mentioned this pull request Apr 21, 2022
43 tasks
Copy link
Copy Markdown
Contributor

@iverase iverase left a comment

Choose a reason for hiding this comment

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

LGTM

@rjernst rjernst merged commit 4d7fc70 into elastic:master Apr 25, 2022
@rjernst rjernst deleted the hppc/perc_agg branch April 25, 2022 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/Aggregations Aggregations >refactoring Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants