External state support? about pybreaker HOT 3 CLOSED

Hey @phillbaker!

Like you said, I'd prefer if such a feature is completely optional.

We could do this by detecting if the required dependencies for some particular storage backend are installed (instead adding these dependencies in setup.py for everybody), kinda like we usually do when dealing with deprecated modules:

# In the module that implements the Redis-based persistence factory, or whatever
try:
  import dependency
except ImportError:
  # Module does not exist, so this feature is not available
  # Raise an error explaining this could only be used if the required dependencies exist

As for the implementation, all I can say is to keep it as simple as possible. It sure looks the implementation in breaker_box is simple enough, so that's fine by me.

from pybreaker.

One more thing is that you need to be careful so that any problems communicating with the storage backend do not interfere with the circuit breaker (and thus not cause application errors) if the storage is offline.

from pybreaker.

Thanks! I'll see where we end up and if it makes sense to PR back to this repo. Thanks for the work on this!

from pybreaker.