Set electionWon upon commit (elastic#16)

The ConsensusState implementation forgot to adapt electionWon on commit.
ywelsch · Dec 19, 2017 · 9d314c4 · 9d314c4
1 parent bfe47ab
commit 9d314c4
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/core/src/main/java/org/elasticsearch/discovery/zen2/ConsensusState.java b/core/src/main/java/org/elasticsearch/discovery/zen2/ConsensusState.java
@@ -267,7 +267,17 @@ public void handleCommit(ApplyCommit applyCommit) {
         persistedState.markLastAcceptedStateAsCommitted();
         assert getCommittedState().getSlot() == applyCommit.getSlot();
         assert getAcceptedState().isPresent() == false;
+
         publishPermitted = true;
+        final boolean prevElectionWon = electionWon;
+        electionWon = isQuorumInCurrentConfiguration(joinVotes);
+        if (prevElectionWon != electionWon) {
+            if (electionWon) {
+                logger.debug("handleCommit: election won through reconfiguration change");
+            } else {
+                logger.debug("handleCommit: election lost through reconfiguration change");
+            }
+        }
         publishVotes = new NodeCollection();
     }