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 waiting for maxDelayRebalance with AckMode.EXACTLY_ONCE during rebalancing #371 #381

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

damn1kk
Copy link

@damn1kk damn1kk commented Jan 21, 2024

Resolves #371

Fixed waiting for maxDelayRebalance with AckMode.EXACTLY_ONCE during rebalancing.

Don't increment uncommitted messages in CommitBatch for AckMode.ExactlyOnce.
Increasing the number of uncommitted messages in CommitBatch causes an infinite loop during rebalancing because they never decrease for AckMode.ExactlyOnce.

@pivotal-cla
Copy link

@damn1kk Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

Don't increment uncommitted messages in CommitBatch for AckMode.ExactlyOnce.
Increasing the number of uncommitted messages in CommitBatch causes an infinite loop during rebalancing because they never decrease.

Fixes reactor#371.
@damn1kk damn1kk force-pushed the fix-long-rebalance-for-exactly-once branch from b25fe21 to a8c7b83 Compare January 22, 2024 18:57
@pivotal-cla
Copy link

@damn1kk Thank you for signing the Contributor License Agreement!

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.

Rebalancing always waits until maxDelayRebalance with AckMode.EXACTLY_ONCE
2 participants