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

Improve dashboard performance / consider simplified top n calculation #2549

Closed
mamoep opened this issue Dec 13, 2023 · 4 comments · Fixed by #2553, #2569 or #2590
Closed

Improve dashboard performance / consider simplified top n calculation #2549

mamoep opened this issue Dec 13, 2023 · 4 comments · Fixed by #2553, #2569 or #2590
Assignees
Labels

Comments

@mamoep
Copy link

mamoep commented Dec 13, 2023

Is your feature request related to a problem? Please describe.
I have 22 clusters monitored and using the SVM dashboard takes up to 20 seconds before I get what I want. A high number of queries in variables for top n calculations.

Describe the solution you'd like
Consider calculating top n once based on volume iops & throughput and then use this value for all protocols.

Describe alternatives you've considered
n/a

Additional context
The approach would be different and my lead to less accurate results, but the usability would improve a lot. Top talkers on the cluster would still be visible.

@mamoep mamoep added the feature New feature or request label Dec 13, 2023
@rahulguptajss
Copy link
Contributor

Discord link.

@cgrinds
Copy link
Collaborator

cgrinds commented Jan 2, 2024

Leaving open to capture remaining work. #2553 fixes SVM dashboard. We should roll this out to all dashboards with topk panels.

@cgrinds
Copy link
Collaborator

cgrinds commented Jan 23, 2024

Nice job! I'll open a new issue for the few remaining vars that were not covered. See also #2599

@rahulguptajss
Copy link
Contributor

verified in main

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