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

observability: allow Span with key for hotkey detection #3

Open
wants to merge 1 commit into
base: instrument-with-opencensus
Choose a base branch
from

Conversation

odeke-em
Copy link
Member

Hotkey detection can now be performed on your backend
by corelating a specific key with a latency.
A hotkey is one that causes a pathologic system
response time e.g. if the key "foo" causes
the system to return in 10ms yet every other
key returns in about 570us.

With a sufficiently expressive filtering backend,
you can ask for the hotest key i.e. with the most
latency.

The last problem though is that we aren't tracking
their frequencies, but that perhaps isn't the point
of tracking hotkeys and has no relation.

The key to filter on is "key" and we can further
drill down by "command" since that's another tag
that's used.

The risk here though is increased latency.

Sample visualization on Stackdriver Monitoring
screen shot 2018-07-12 at 11 26 47 pm

Hotkey detection can now be performed on your backend
by corelating a specific key with a latency.
A hotkey is one that causes a pathologic system
response time e.g. if the key "foo" causes
the system to return in 10ms yet every other
key returns in about 570us.

With a sufficiently expressive filtering backend,
you can ask for the hotest key i.e. with the most
latency.

The last problem though is that we aren't tracking
their frequencies, but that perhaps isn't the point
of tracking hotkeys and has no relation.

The key to filter on is "key" and we can further
drill down by "command" since that's another tag
that's used.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant