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

Cherry-pick PRs for 2.10.15-RC.5 #5376

Merged
merged 7 commits into from May 1, 2024
Merged

Conversation

jnmoyne and others added 5 commits May 1, 2024 08:18
- Resets the stalling detection timestamp upon re-creating the consumer
- Adjust the idle heart beat for source/mirror consumers from 1s to 5s

---------

Signed-off-by: Jean-Noël Moyne <jnmoyne@gmail.com>
- Add comments to document main structure field names a bit better and
improve the code accessibility.
- Rename setSourceConsumer to setupSourceConsumer (better and matches
setupMirrorConsumer).

Signed-off-by: Jean-Noël Moyne <jnmoyne@gmail.com>

---------

Signed-off-by: Jean-Noël Moyne <jnmoyne@gmail.com>
…orConsumer (#5369)

Move cancelMirrorConsumer call inside the locked section in
retryMirrorConsumer

Signed-off-by: Jean-Noël Moyne <jnmoyne@gmail.com>
Signed-off-by: Derek Collison <derek@nats.io>
Signed-off-by: Derek Collison <derek@nats.io>
@wallyqs wallyqs requested a review from a team as a code owner May 1, 2024 15:23
@wallyqs wallyqs changed the title Cherry picks for release/v2.10.15 Cherry-pick PRs for 2.10.15-RC.5 May 1, 2024
jnmoyne and others added 2 commits May 1, 2024 13:02
…t and discard new.

- Avoids re-scanning the stream to find the consumer's starting sequence number for that source every time. Do not log a warning every time this happens as it becomes constant log spam for something that happens temporarily all the time.

- Strengthen test TestJetStreamClusterSourceWorkingQueueWithLimit to also check the payload (to catch for example the same message being sourced more than once, or messages not getting sourced).

Signed-off-by: Jean-Noël Moyne <jnmoyne@gmail.com>
…che expiration logic.

Also added in additional filestore config to control this if needed, still defaults to 10s.

Signed-off-by: Derek Collison <derek@nats.io>
Copy link
Member

@bruth bruth left a comment

Choose a reason for hiding this comment

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

LGTM

@wallyqs wallyqs merged commit 9d29e8d into release/v2.10.15 May 1, 2024
2 of 4 checks passed
@wallyqs wallyqs deleted the downstream/v2.10.15 branch May 1, 2024 20:10
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

4 participants