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

[FIXED] JetStream: Ordered consumers may not be reset in some cases #596

Merged
merged 1 commit into from
Oct 3, 2022

Conversation

kozlovic
Copy link
Member

@kozlovic kozlovic commented Oct 1, 2022

If the recreation of the ordered consumer were to fail, the library would invalidate the subscription and not try again.

This PR also changes the ordered consumer to use memory storage and replicas of 1. It also ensures that the ordered consumer is reset on missed heartbeats.

Signed-off-by: Ivan Kozlovic [email protected]

If the recreation of the ordered consumer were to fail, the library
would invalidate the subscription and not try again.

This PR also changes the ordered consumer to use memory storage
and replicas of 1. It also ensures that the ordered consumer is
reset on missed heartbeats.

Signed-off-by: Ivan Kozlovic <[email protected]>
@kozlovic kozlovic requested a review from aricart October 1, 2022 01:20
Copy link
Member

@aricart aricart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kozlovic kozlovic merged commit 8a04792 into main Oct 3, 2022
@kozlovic kozlovic deleted the fix_ordered_consumers branch October 3, 2022 15:30
kozlovic added a commit that referenced this pull request Oct 4, 2022
- Was not setting to MemoryStorage and Replicas of 1 on recreate
- Added test that checks for MemoryStorage after a reset
- Improved error reporting when failing to recreate the consumer

Related to #596

Signed-off-by: Ivan Kozlovic <[email protected]>
kozlovic added a commit that referenced this pull request Oct 4, 2022
- Was not setting to MemoryStorage and Replicas of 1 on recreate
- Added test that checks for MemoryStorage after a reset
- Improved error reporting when failing to recreate the consumer

Related to #596

Signed-off-by: Ivan Kozlovic <[email protected]>
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.

2 participants