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] Clustering: possible wrong pending_count and is_stalled on followers #1224

Merged
merged 2 commits into from Nov 17, 2021

Conversation

kozlovic
Copy link
Member

If a queue member receives a message and does not ack it within
the AckWait interval, this message may be redelivered to a different
member. If the original member then acknowledge the message, the
server finds it in the other member and remove it from the pending
list. This would cause the second member to show a wrong pending_count
and possibly is_stalled values in monitoring of the followers.

Signed-off-by: Ivan Kozlovic ivan@synadia.com

…llowers

If a queue member receives a message and does not ack it within
the AckWait interval, this message may be redelivered to a different
member. If the original member then acknowledge the message, the
server finds it in the other member and remove it from the pending
list. This would cause the second member to show a wrong pending_count
and possibly is_stalled values in monitoring of the followers.

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
Copy link
Member

@derekcollison derekcollison left a comment

Choose a reason for hiding this comment

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

LGTM

A change in v0.10.1 caused failed Close() to keep the underlying
NATS connection opened, which would cause some tests to fail
because the NATS connection would reconnect to subsequent tests,
etc..

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 91.451% when pulling 78d6066 on cluster_queue_rdlv into 4f5a913 on main.

@kozlovic kozlovic merged commit 0888c7d into main Nov 17, 2021
@kozlovic kozlovic deleted the cluster_queue_rdlv branch November 17, 2021 20:39
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.

None yet

3 participants