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

Dispatch queue backpressure and batching #1118

Merged
merged 2 commits into from Apr 28, 2023
Merged

Conversation

dranikpg
Copy link
Contributor

@dranikpg dranikpg commented Apr 20, 2023

Fixes #1039

This PR:

  • Adds "almost correct" backpressure to publish messages (without decreasing dispatch performance)
  • Enables batching for pubsub

@dranikpg dranikpg changed the title [WIP] Dispatch queue backpressure [WIP] Dispatch queue backpressure and batching Apr 20, 2023
@dranikpg dranikpg force-pushed the dsq-bp branch 4 times, most recently from d68baa6 to 5ee9c62 Compare April 22, 2023 07:22
Signed-off-by: Vladislav Oleshko <vlad@dragonflydb.io>
@dranikpg dranikpg marked this pull request as ready for review April 22, 2023 07:25
@dranikpg dranikpg changed the title [WIP] Dispatch queue backpressure and batching Dispatch queue backpressure and batching Apr 22, 2023
@dranikpg dranikpg requested a review from romange April 23, 2023 08:03
Signed-off-by: Vladislav Oleshko <vlad@dragonflydb.io>
@dranikpg dranikpg requested a review from romange April 24, 2023 06:46
@dranikpg dranikpg merged commit 432ece3 into dragonflydb:main Apr 28, 2023
6 checks passed
@dranikpg dranikpg deleted the dsq-bp branch April 30, 2023 15:38
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

Successfully merging this pull request may close these issues.

No backpressure on pubsub and monitor leads to piling up messages
2 participants