Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[APM] Replace top_hits with top_metrics aggregation #90910

Closed
dgieselaar opened this issue Feb 10, 2021 · 1 comment · Fixed by #91479
Closed

[APM] Replace top_hits with top_metrics aggregation #90910

dgieselaar opened this issue Feb 10, 2021 · 1 comment · Fixed by #91479
Assignees
@dgieselaar
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:APM All issues that need APM UI Team support v7.12.0

Comments

@dgieselaar
Copy link
Member

We are currently using top_hits to get sample document values in bucketing aggregations, e.g. getting agent.name per service. We should replace these with a top_metrics aggregation which is faster.
@dgieselaar dgieselaar added Team:APM All issues that need APM UI Team support release_note:skip Skip the PR/issue when compiling release notes v7.12.0 labels Feb 10, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

This was referenced Feb 10, 2021
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dgieselaar dgieselaar

Labels
release_note:skip Skip the PR/issue when compiling release notes Team:APM All issues that need APM UI Team support v7.12.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[APM] use top_metrics aggregation where appropriate dgieselaar/kibana
3 participants
@graphaelli @dgieselaar @elasticmachine