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.

Sign up for GitHub

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

Jump to bottom

Pubsub can end up blocked by subscriptions with no subscribers #1685

Closed

Tracked by

matt2e opened this issue Jun 6, 2024 · 0 comments · Fixed by #1752

Closed

Tracked by

Pubsub can end up blocked by subscriptions with no subscribers #1685

matt2e opened this issue Jun 6, 2024 · 0 comments · Fixed by #1752

Assignees

Collaborator

matt2e commented Jun 6, 2024

Have 5 subscriptions with no subscribers
Publish events to each subscription's topic
Now when controllers try to progress subscriptions, they will always get these subscriptions because they have the oldest cursor and we limit by 3
controllers try to progress 3 of these subscriptions, which becomes a no-op because there are no subscribers
... repeat indefinitely, so we never actually progress subscriptions with newer cursors and actual subscribers

matt2e self-assigned this

matt2e mentioned this issue

Closed

14 tasks

ftl-robot mentioned this issue

Open

matt2e mentioned this issue

fix: remove old subscriptions and subscribers #1752

Merged

matt2e closed this as completed in #1752

matt2e closed this as completed in

22f94f7

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