-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[APM] Use _doc_count
from service metrics
#141210
Conversation
Pinging @elastic/apm-ui (Team:APM) |
@dgieselaar what's best release label, can I still add it as part of 8.5? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
.../plugins/apm/server/routes/services/get_services/get_service_aggregated_transaction_stats.ts
Show resolved
Hide resolved
...s/get_services_detailed_statistics/get_service_aggregated_transaction_detailed_statistics.ts
Show resolved
Hide resolved
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
_doc_count
from transaction metrics_doc_count
from service metrics
* [APM] Use `_doc_count` from transaction metrics * Remove unnecessary agg (cherry picked from commit 926fd12)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
* [APM] Use `_doc_count` from transaction metrics * Remove unnecessary agg (cherry picked from commit 926fd12) Co-authored-by: Katerina Patticha <aikaterini.patticha@elastic.co>
part of: #141185
elastic/apm-server#9143
apm-server added the
_doc_count
to ensure proper bucket counts and avoid the aggregation on value count.