HystrixCircuitBreakerImpl isOpen can return false incorrectly

[mswintermeyer]

At the end of the method, after the compareAndSet, it should always be return true. compareAndSet sets the value to the given updated value if the current value == the expected value. If the method returns false, it means that the current value is already true, which means the circuit is open (another thread got here first in a race) and we should return true.

[mattrjacobs]

Thanks for the report @mswintermeyer , I'll look into it

[mattrjacobs]

Yes, I agree with your fix. Nice catch! I'm getting a PR ready right now...