search

spring-cloud / spring-cloud-circuitbreaker

Spring Cloud Circuit Breaker API and Implementations
Apache License 2.0
336 stars 110 forks source link

ReactiveResilience4JCircuitBreaker Missing Bulkhead Support #105

Open ryanjbaxter opened 3 years ago

ryanjbaxter commented 3 years ago

Bulkhead support was added in the non-reactive case in https://github.com/spring-cloud/spring-cloud-circuitbreaker/issues/93

We should also support it in the reactive case

ryanjbaxter commented 3 years ago

@Ferioney would you be interested in adding support?

Ferioney commented 3 years ago

@ryanjbaxter yes, I can handle this. I think for the reactive client we should use just Semaphorne bulkhead. (which is non-blocking). What do you think?

ryanjbaxter commented 3 years ago

Sounds good, thanks!

jwalgemoed commented 2 years ago

Hi @Ferioney, is this still a planned enhancement? It seems to have been pushed forward quite some time. Is there something I could do to get this implemented? A branch I can look at? We need this feature so I’d be happy to try and assist in adding it.