Committing entries from previous terms about raft HOT 8 CLOSED

Thanks for logging this issue, good catch!

The quorum logic should implement the part in Rules for Servers - Leaders at the bottom of figure 2 in raft extended

If there exists an N such that N > commitIndex, 
a majority of matchIndex[i] ≥ N, 
and log[N].term == currentTerm 
: set commitIndex = N.

This does not seem to be the case in our implementation.

from raft.

Any plan to fix this？

from raft.

Yes, it is still the plan to fix this.

from raft.

This bugs find almost a year, Is there a schedule plan for it.

from raft.

I have a branch where I'm implementing a fix as it has some unintended consequences in dqlite, however, unfortunately, things keep coming up in between to finish it.

from raft.

Thanks for the quick response, look forward to it.

If you can share your branch here, we can try it and maybe give some advice.

from raft.

https://github.com/MathieuBordere/raft/commits/quorum-bugfix it's behind the current master somewhat though.

from raft.

I'm in the process of picking up Mathieu's previous work on this, will have a draft PR up soon.

from raft.