-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Support for Half-Open state in Circuit Braker pattern #1459
Comments
I'm not a Hystrix dev but, the half-open state you describe is actually pretty much what Hystrix already does: https://github.com/Netflix/Hystrix/wiki/How-it-Works#circuit-breaker |
@56quarters is correct - this is already the way it works. |
How do you specify how many requests should be forwarded to the service during the Half Open state? |
It's hardcoded to do a single request after the sleep window elapses. If It succeeds, it transitions to CLOSED. If it fails, it transitions to OPEN and goes back to sleep for the specified amount of time. |
Closing due to inactivity. Please re-open if there's more to discuss. |
More than two years have passed. Does Hystrix still hardcoded a single request? In my practice, the 1.5.18 version still do this. Looking forward to your reply. Thank you. |
@mattrjacobs How can we hardcode the no of requests we want after the sleep window elapses? Also this means the run method of the command would get executed those many times right? |
Hi,
It's my understanding that the current version of the Circuit Braker pattern implemented in the Netflix library supports only two states:
It would be great to add a middle state (Half Open) as described in the following link .
Quoting the description of the Half Open State:
The text was updated successfully, but these errors were encountered: