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

Statistics getTotalGetLatency, getTotalRemoveLatency, getTotalPutLatency works incorrectly #10938

Closed
ihsandemir opened this issue Jul 17, 2017 · 0 comments

Comments

@ihsandemir
Copy link
Contributor

@ihsandemir ihsandemir commented Jul 17, 2017

Local statistics uses milliseconds for get/put/remove latencies. It is very usual for latencies to be around microseconds, hence the values may become 0 very easily. Especially the accumulative statistics such as totalGetLatencies and such, it does not make sense to stay at 0 even though there are millions of gets each below millisecond latency.

E.g. Our get latency for a single get is usually in microseconds around 100-200usecs. If you do 1 million get, then we normally expect the latency to be 1 million * 200 usec=200 sec. But since latency for each request is less than a millisecond, it is recorded as to be zero and the total becomes zero.

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

Successfully merging a pull request may close this issue.

2 participants
You can’t perform that action at this time.