Skip to content

Move scripted_metric agg to a plugin #19821

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
colings86 opened this issue Aug 5, 2016 · 4 comments
Closed

Move scripted_metric agg to a plugin #19821

colings86 opened this issue Aug 5, 2016 · 4 comments
Labels
:Analytics/Aggregations Aggregations help wanted adoptme Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)

Comments

@colings86
Copy link
Contributor

The scripted_metric aggregation is an expert level feature and has its own set of tradeoffs that need to be considered before reaching for it. Due to the fact that it uses scripts it is slow (when compared to other aggregations) especially as the complexity of the scripts increase. It also is not able to support certain functions, for example, you cannot sort a terms aggregation by the scripted_metric aggregation.

I think we should therefore move the scripted_metric aggregation to a plugin (not a module) so when users reach for it, it is a more explicit that this is an expert level aggregation

@polyfractal
Copy link
Contributor

@elastic/es-search-aggs

@colings86 colings86 added help wanted adoptme and removed discuss labels Apr 30, 2018
@rjernst rjernst added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label May 4, 2020
@talevy
Copy link
Contributor

talevy commented Nov 9, 2020

Hi @colings86 it's been a while for this issue. Do you think this is still worth doing?

@talevy
Copy link
Contributor

talevy commented Nov 9, 2020

@jimczi let me know that there is a similar issue that there is a duplicate issue for this opened later. Since that issue, although a duplicate, has had more cross-references from other projects (kibana and grafana), then I'll close this in favor of #63096

@talevy talevy closed this as completed Nov 9, 2020
@polyfractal
Copy link
Contributor

Whoops, I didn't realize we had this one open when I made the other. My bad!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Aggregations Aggregations help wanted adoptme Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)
Projects
None yet
Development

No branches or pull requests

4 participants