Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

Fix last_applied failing assertion #423

Merged
merged 2 commits into from
Jun 2, 2023
Merged

Fix last_applied failing assertion #423

merged 2 commits into from
Jun 2, 2023

Commits on Jun 1, 2023

  1. test/integration/replication: Add lastStoredLaggingBehindCommitIndex 

    Add a failing test that reproduces the situation that triggered the assertion
failure described in #355.

Signed-off-by: Free Ekanayaka <free@ekanayaka.io>
    @freeekanayaka
    freeekanayaka committed Jun 1, 2023
    Configuration menu
    Copy the full SHA
    c553bae View commit details
    Browse the repository at this point in the history

  2. replication: Only update commit index if last stored is high enough 

    A leader might have its last_stored field lagging behind its commit_index,
after it committed an index that it did not yet persist, while enough followers
for forming a majority did.

We don't wont to update this leader's commit_index until last_stored has caught
up.

Signed-off-by: Free Ekanayaka <free@ekanayaka.io>
    @freeekanayaka
    freeekanayaka committed Jun 1, 2023
    Configuration menu
    Copy the full SHA
    d144504 View commit details
    Browse the repository at this point in the history