-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Design latency monitoring for DiceDB #1135
Comments
@swarajrb7 would you be interested in taking this up? |
Yes |
Hey, it may be good to discuss the design before you begin working on the implementation. We need to ensure this does not introduce any performance regressions on the critical path. |
I am researching on how this is implemented in Redis. I do have some questions about it. I will let you know as soon as possible. |
@JyotinderSingh can we discuss this today? i have send a message on discord channel |
I am attaching a file in which i have written down my observation and my queries. Please take a look and give me suggestions/improvement to make. |
We need to support the different
LATENCY
commands which redis supports. This requires quite a few changes to the codebase.Your task as a part of this issue would be to design a latency monitoring approach for dicedb, the approach should allow us to serve all the different latency commands:
LATENCY RESET
LATENCY HISTORY
LATENCY LATEST
LATENCY HISTOGRAM
(low priority)LATENCY GRAPH
(low priority)The text was updated successfully, but these errors were encountered: