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

Single event handler invocation queue per session #74

Closed
yallie opened this issue Aug 7, 2020 · 1 comment
Closed

Single event handler invocation queue per session #74

yallie opened this issue Aug 7, 2020 · 1 comment
Milestone

Comments

@yallie
Copy link
Member

yallie commented Aug 7, 2020

The current implementation uses a queue per subscription.
When a session initiates many subscriptions and doesn't unsubscribe
properly, it creates lots of work threads on the server.

Invocation queue is implemented as a thread pool.
Per-subscription queue is a pool that uses a single thread.
Per-session queue will probably need more than one thread (should be adjustable).

@yallie yallie added this to the 2.14 milestone Aug 7, 2020
@yallie
Copy link
Member Author

yallie commented Aug 10, 2020

Implemented in df91c1a.

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

1 participant