-
Notifications
You must be signed in to change notification settings - Fork 62
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
(WIP) Add @circuit_breaker decorator #156
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #156 +/- ##
===========================================
- Coverage 100% 86.93% -13.07%
===========================================
Files 41 42 +1
Lines 2122 2441 +319
Branches 167 177 +10
===========================================
Hits 2122 2122
- Misses 0 319 +319
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #156 +/- ##
===========================================
- Coverage 100% 87.07% -12.93%
===========================================
Files 41 42 +1
Lines 2175 2468 +293
Branches 173 182 +9
===========================================
- Hits 2175 2149 -26
- Misses 0 319 +319
Continue to review full report at Codecov.
|
I can think of several instances, recently, where a circuit breaker would have been very helpful due to a misbehaving service that was giving itself the DDoS treatment. :p Very Cool! |
Changes proposed in this pull request:
@circuit_breaker
decorator: decorating aConsumer
subclass with this annotation adds a Circuit Breaker that monitors request(s) and improves the resilience of the client against cascading failures caused by downtime of the consumed service.