awolden / brakes

Hystrix compliant Node.js Circuit Breaker Library

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add Cumulative Counts to Hystrix Stream

SimenB opened this issue · comments

Currently a rolling count is kept, but not a cumulative one.

It would be great of both were available.

https://github.com/Netflix/Hystrix/wiki/Metrics-and-Monitoring#cumulative-and-rolling-event-counts

This would be a solid addition! Flagging as a feature request.

Please say here if you work on it, I have it on my TODO list (which is admittedly pretty long ATM).

@awolden any update on this one? keen to assist if no on else is

I haven't gotten to it yet

@SimenB @awolden I worked on that and created a PR, would be great if you guys could review and merge it. I think it is an important enhancement for this lib.

PR: #112

Thanks for the contribution @Floix!