X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-akka-raft%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fraft%2Fbehaviors%2FCandidate.java;h=7f73d0cfb0a24a8b85960c2b1fdb031c3131cfdd;hb=01c5a7cc52f8a438053d52ebb91e531493ca765a;hp=a59a02051a7a8e5e5563429b098de55284829871;hpb=c68d251880d95d6d2f8df70c67d2cdd3a3a47685;p=controller.git diff --git a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/behaviors/Candidate.java b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/behaviors/Candidate.java index a59a02051a..7f73d0cfb0 100644 --- a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/behaviors/Candidate.java +++ b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/behaviors/Candidate.java @@ -10,7 +10,9 @@ package org.opendaylight.controller.cluster.raft.behaviors; import akka.actor.ActorRef; import akka.actor.ActorSelection; -import java.util.Set; +import java.util.ArrayList; +import java.util.Collection; +import org.opendaylight.controller.cluster.raft.PeerInfo; import org.opendaylight.controller.cluster.raft.RaftActorContext; import org.opendaylight.controller.cluster.raft.RaftState; import org.opendaylight.controller.cluster.raft.base.messages.ElectionTimeout; @@ -44,22 +46,26 @@ public class Candidate extends AbstractRaftActorBehavior { private final int votesRequired; - private final Set peers; + private final Collection votingPeers = new ArrayList<>(); public Candidate(RaftActorContext context) { super(context, RaftState.Candidate); - peers = context.getPeerAddresses().keySet(); + for(PeerInfo peer: context.getPeers()) { + if(peer.isVoting()) { + votingPeers.add(peer.getId()); + } + } if(LOG.isDebugEnabled()) { - LOG.debug("{}: Election: Candidate has following peers: {}", logName(), peers); + LOG.debug("{}: Election: Candidate has following voting peers: {}", logName(), votingPeers); } - votesRequired = getMajorityVoteCount(peers.size()); + votesRequired = getMajorityVoteCount(votingPeers.size()); startNewTerm(); - if(context.getPeerAddresses().isEmpty()){ + if(votingPeers.isEmpty()){ actor().tell(ELECTION_TIMEOUT, actor()); } else { scheduleElection(electionDuration()); @@ -170,7 +176,7 @@ public class Candidate extends AbstractRaftActorBehavior { // Request for a vote // TODO: Retry request for vote if replies do not arrive in a reasonable // amount of time TBD - for (String peerId : peers) { + for (String peerId : votingPeers) { ActorSelection peerActor = context.getPeerActorSelection(peerId); if(peerActor != null) { RequestVote requestVote = new RequestVote(