failsafe-lib / failsafe

Fault tolerance and resilience patterns for the JVM
https://failsafe.dev
Apache License 2.0
4.16k stars 295 forks source link

Changed BulkheadImpl.releasePermit to ensure that it claims a permit when running a task from the pending queue #366

Closed nicky9door closed 1 year ago

nicky9door commented 1 year ago

Before this change, releasePermit would check the queue but not modify the permit field. This could eventually result in permits being equal to maxPermits and would erroneously report the bulkhead as being full

This should fix issue #365

jhalterman commented 1 year ago

Thanks @nicky9door !