[RATIS-1256] Leader updateCommit should use the updated index

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.0.0
• Component/s: server
• Assignee: Tsz-wo Sze
• Reporter: Tsz-wo Sze
• Created: 21/Dec/20 05:35
• Updated: 19/May/22 20:36
• Resolved: 22/Dec/20 02:41

Description

In LeaderStateImpl.updateCommit, it gets entriesToCommit using the majority index and then updateCommitIndex. However, updateCommitIndex may update to a lower index than the majority index. Therefore, entriesToCommit may contain non-committed entries.

Issue Links

• links to: GitHub Pull Request #369