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

Shared Message Dispatcher thread pool #141

Open
danduff opened this issue Jul 22, 2016 · 1 comment
Open

Shared Message Dispatcher thread pool #141

danduff opened this issue Jul 22, 2016 · 1 comment

Comments

@danduff
Copy link

danduff commented Jul 22, 2016

I'm creating a number of Async EventBuses.
The HandlerInvocation has the option to provide it with a threadpool (which I have shared across all EventBuses), however for every Bus created a new Dispatch thread is created which is specific to each Bus. This means I can't have a fixed limit of thread creation other than limiting the number of Buses that are created.
Is there a reason why the Dispatcher can't use a thread pool which could be shared the same way as the HandlerInvocation?

@bennidi
Copy link
Owner

bennidi commented Jul 22, 2016

No there isn't. This requirement simply did not occur to me. You are welcome to provide a PR that enables this feature. P.S. Only well-tested code is accepted :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants