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

[APM] Service Maps: Duration and req/min are average values across all transaction types #66704

Closed
sorenlouv opened this issue May 15, 2020 · 4 comments
Assignees
Labels
apm:service-maps Service Map feature in APM needs design Team:APM All issues that need APM UI Team support v7.9.0

Comments

@sorenlouv
Copy link
Member

sorenlouv commented May 15, 2020

Currently the Service Maps popover shows duration (637ms) and throughput (7.35rpm) across all transaction types:

I'm not sure that having an average duration of 637ms says anything since it's a mix of background tasks (average of 1016ms) and requests (average of 91ms).

Proposal 1
Only show duration and throughput for "type=request".

Proposal 2
Show duration and throughput per type

Proposal 3
Add transaction type selector and show duration and throughput for the selected type

@sorenlouv sorenlouv added apm:service-maps Service Map feature in APM Team:APM All issues that need APM UI Team support labels May 15, 2020
@elasticmachine
Copy link
Contributor

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

@sorenlouv sorenlouv removed the v7.8.0 label May 15, 2020
@sorenlouv sorenlouv changed the title [APM] Service Maps: Consider scoping to only transaction.type=request [APM] Service Maps: Duration and req/min are average values across all transaction types May 19, 2020
@formgeist
Copy link
Contributor

@sqren This is no longer relevant unless we feel it's the same possible confusion with different environment metrics together? I think the better option, as we've discussed, is to not have an All environments option available, but simply pick the first in the list (by amount of records) and let the user switch using the picker?

@sorenlouv
Copy link
Member Author

sorenlouv commented Jul 1, 2020

Unless we introduce a transaction type selector (or one of the other proposals), this will still be an issue.

@formgeist
Copy link
Contributor

As we agreed to move forward with proposal 3 ("Add transaction type selector and show duration and throughput for the selected type"), I've created #70419 to replace this issue and the improved design.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apm:service-maps Service Map feature in APM needs design Team:APM All issues that need APM UI Team support v7.9.0
Projects
None yet
Development

No branches or pull requests

3 participants