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.

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

Add event.outcome to metrics aggregation #4041

Closed
dgieselaar opened this issue Jul 30, 2020 · 0 comments · Fixed by #4110
Closed

Add event.outcome to metrics aggregation #4041

dgieselaar opened this issue Jul 30, 2020 · 0 comments · Fixed by #4110
Assignees
Milestone

Comments

@dgieselaar
Copy link
Member

dgieselaar commented Jul 30, 2020

See elastic/apm#299

We'll be adding event.outcome to transaction docs, which (per https://www.elastic.co/guide/en/ecs/current/ecs-allowed-values-event-outcome.html) has a limited set of values: success, failure, unknown, (unset)

This field will be used for error rates. So that we can use aggregated metrics docs for the calculations, we will need to include this field in transaction metrics docs, which also implies that it needs to be included in the aggregation key. This should have no or little impact on series cardinality, as event.outcome will typically be subsumed by transaction.result.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants