-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add support for the LATENCY and MEMORY commands #1204
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some notes on args but looking good 👍
I merged master in to get builds going (another image break). Code looks good - I'll take a look at some memory tests in the AM then good to go I think - nice work man! |
Note for tomorrow: the 2 latency tests are failing on Appveyor, need to look into those |
Finally dug into this one @mgravell, turns out we just didn't have a threshold set (and I don't see this as conflicting with other tests).
Looking good now - merging in! |
Adds LATENCY and MEMORY command support and testing.
Adds LATENCY and MEMORY command support and testing.
Adds LATENCY and MEMORY command support and testing.
Adds LATENCY and MEMORY command support and testing.
Adds LATENCY and MEMORY command support and testing.