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

backpressure for matching topics? #5

Closed
behrad opened this issue Jul 4, 2017 · 2 comments
Closed

backpressure for matching topics? #5

behrad opened this issue Jul 4, 2017 · 2 comments

Comments

@behrad
Copy link
Contributor

behrad commented Jul 4, 2017

Does it help if we change this to a streamable one? Does it provide back pressure when publishing a message to a wildcard topic with too many subscribers @mcollina ?

@mcollina
Copy link
Collaborator

mcollina commented Jul 4, 2017

The problem is that the cost is already paid, which is in the creation big array. Even if it is streamable, it will not help as the data structure as already been allocated. However, we can change https://github.com/mcollina/aedes/blob/e347031544512f23999a54df4debdae08628d5b1/aedes.js#L194-L196 to be a queue and not act on all of them in parallel.

@behrad
Copy link
Contributor Author

behrad commented Aug 11, 2017

moscajs/aedes#131

@behrad behrad closed this as completed Aug 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants