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

Update consumer last_seen on status messages and requests #856

Merged
merged 2 commits into from
Mar 1, 2023

Conversation

Jarema
Copy link
Member

@Jarema Jarema commented Mar 1, 2023

Getting any message from consumer, even timeout, should reset the last_seen timer.

Signed-off-by: Tomasz Pietrek tomasz@nats.io

@Jarema Jarema requested a review from caspervonb March 1, 2023 15:58
@Jarema Jarema force-pushed the jarema/improve-idle-hearbeat-handling branch from f90de6f to 4e537db Compare March 1, 2023 20:02
Getting any message from consumer, even timeout, should reset
the last_seen timer.

Signed-off-by: Tomasz Pietrek <tomasz@nats.io>
Idle heartbeat should not terminate iterator, while
status of consumer deleted should.

Signed-off-by: Tomasz Pietrek <tomasz@nats.io>
@Jarema Jarema force-pushed the jarema/improve-idle-hearbeat-handling branch from 4e537db to 223d01b Compare March 1, 2023 20:43
Copy link
Collaborator

@caspervonb caspervonb left a comment

Choose a reason for hiding this comment

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

lgtm

@Jarema Jarema merged commit 68f4b7c into main Mar 1, 2023
@Jarema Jarema deleted the jarema/improve-idle-hearbeat-handling branch March 1, 2023 22:33
@Jarema Jarema mentioned this pull request Mar 1, 2023
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

2 participants