Skip to content
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

ReactiveResilience4JCircuitBreaker Missing Bulkhead Support #105

Open
ryanjbaxter opened this issue Apr 8, 2021 · 4 comments
Open

ReactiveResilience4JCircuitBreaker Missing Bulkhead Support #105

ryanjbaxter opened this issue Apr 8, 2021 · 4 comments
Labels
enhancement New feature or request

Comments

@ryanjbaxter
Copy link
Contributor

Bulkhead support was added in the non-reactive case in #93

We should also support it in the reactive case

@ryanjbaxter ryanjbaxter added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels Apr 8, 2021
@ryanjbaxter ryanjbaxter added this to the 1.0.4.RELEASE milestone Apr 8, 2021
@ryanjbaxter
Copy link
Contributor Author

@Ferioney would you be interested in adding support?

@Ferioney
Copy link

@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
Copy link
Contributor Author

Sounds good, thanks!

@jwalgemoed
Copy link

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants