Skip to content

Replication bug fixes #516

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

Merged
merged 2 commits into from
Mar 12, 2025
Merged

Replication bug fixes #516

merged 2 commits into from
Mar 12, 2025

Conversation

kjnilsson
Copy link
Contributor

@kjnilsson kjnilsson commented Mar 7, 2025

  1. Fix bug where occasionally one follower would receive
    more entries then expected resulting in triggering 2.

  2. Fix bug when follower received an append_entries_rpc with
    no new entries in it would trigger an incorrect resend
    scenario when throughput is high.

  3. Fix bug where a late written event could accidentally confirm
    unwritten entries in a higher term.

  4. A replication liveness bug that could cause clusters to get stuck
    after a member removal.

@kjnilsson kjnilsson force-pushed the follower-resend-bug branch 3 times, most recently from 52d7fe5 to f4460a5 Compare March 7, 2025 16:05
1. Fix bug where occasionally one follower would receive
more entries then expected resulting in triggering 2.

2. Fix bug when follower received an append_entries_rpc with
no new entries in it would trigger an incorrect resend
scenario when throughput is high.

3. Fix bug where a late written event could accidentally confirm
unwritten entries in a higher term.
@kjnilsson kjnilsson force-pushed the follower-resend-bug branch from f1b3784 to 324d9bc Compare March 11, 2025 07:56
@kjnilsson kjnilsson marked this pull request as ready for review March 11, 2025 08:03
@kjnilsson kjnilsson changed the title Follower resend bug Replication bug fixes Mar 11, 2025
@kjnilsson kjnilsson modified the milestones: 2.16.3, 2.15.3 Mar 11, 2025
@kjnilsson
Copy link
Contributor Author

This should be backported to the 2.15.x branch and a new 2.15.x release should be created.

@kjnilsson kjnilsson added the bug label Mar 11, 2025
@kjnilsson kjnilsson force-pushed the follower-resend-bug branch 4 times, most recently from 3f392e6 to 73b73af Compare March 12, 2025 11:21
Add assertions

Add some logging and a test
@kjnilsson kjnilsson force-pushed the follower-resend-bug branch from 73b73af to 60c2d19 Compare March 12, 2025 11:50
@michaelklishin michaelklishin merged commit a9a4897 into main Mar 12, 2025
3 checks passed
@michaelklishin
Copy link
Contributor

@Mergifyio backport v2.15.x

Copy link

mergify bot commented Mar 12, 2025

backport v2.15.x

✅ Backports have been created

kjnilsson added a commit that referenced this pull request Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants