Code Review / controller.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 10ef530) | patch
Bug in AbstractLeader replication consensus 99/33099/1
authorGary Wu <gary.wu1@huawei.com>
Fri, 15 Jan 2016 19:31:48 +0000 (11:31 -0800)
committerTom Pantelis <tpanteli@brocade.com>
Wed, 20 Jan 2016 14:08:43 +0000 (14:08 +0000)
commit6ee773bba778ce80a1008875f0a3d28b81817676
tree284f2a0d50b34e133e10434ae66a8a5f04e791d5tree | snapshot
parent10ef5303f76d27271def9296d511916f26b6649ccommit | diff
Bug in AbstractLeader replication consensus

In determining whether to advance the commit index, only the voting
members should be counted in the replicatedCount.  There was a logic
error that instead caused it to be incorrectly based on whether the
AppendEntriesReply message as sent by a voting member.

This patch fixes the issue.

Change-Id: I6efb9574c39db608351297fc2552689d1ff77979
Signed-off-by: Gary Wu <gary.wu1@huawei.com>
opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/behaviors/AbstractLeader.java diff | blob | history
opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/behaviors/LeaderTest.java diff | blob | history
OpenDaylight controller