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

Update scylla_io_queue_total_delay_sec metrics dispay (if possible) #1714

Closed
xemul opened this issue Apr 6, 2022 · 2 comments · Fixed by #1732
Closed

Update scylla_io_queue_total_delay_sec metrics dispay (if possible) #1714

xemul opened this issue Apr 6, 2022 · 2 comments · Fixed by #1732
Labels
enhancement New feature or request

Comments

@xemul
Copy link

xemul commented Apr 6, 2022

After fixing the #1570 there appeared an inconvenience. The problem is that the ..._total_delay_sec metrics shows the sum of individual requests delays over some priod of time. And, for example, if there had passed 1k of requests with 50us delay each the metrics would report delta of 1k * 50us = 50ms. Thus when looking at the metrics it's always needed to make the reverse division (reported delay / reported IOPS) to get the idea of individual request latency.

If that's possible the dashboard should draw the rate(total_delay_sec) / rate(IOPS).

@amnonh
Copy link
Collaborator

amnonh commented May 15, 2022

@xemul is this only relevant to 5.0 and 2022.1 ?

@xemul
Copy link
Author

xemul commented May 16, 2022

No, the ..._total_delay_sec appeared earlier. It's exported in 4.5.3 and 4.5.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants