-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Feature Request - Include timestamp of 1st failure event in CircuitBreakerOnFailureRateExceededEvent #2145
Comments
Hi, |
Thanks for getting back! Even if clients rely on maintaining a record of failure events, they still will need to maintain a sliding window to determine the first failure in the window. Am I understanding it right? |
That's a good point. |
We just need to keep track of timestamp of first reported failure, if any, for every second in the sliding window and include this detail when creating a snapshot. CircuitBreaker.Metrics will expose a method to get details about failure timestamps (whether getting the earliest reported failure timestamp or similar). When publishing rate exceeded events, timestamp detail could be included: Line 951 in dc9505a
What else is needed? |
It may be better to only expose earliest failure timestamp on demand i.e. only when generating error events. |
I'd like the ability to know how long it takes for Circuit Breaker to trip:
T2 - T1
If this data was made available through
CircuitBreakerOnFailureRateExceededEvent
, it would be perfect for clients to record how long detection took. Let me know what you think.I couldn't find a similar request from the past; please let me know if I missed any.
The text was updated successfully, but these errors were encountered: