Browse Source

Merge pull request #1846 from xiang90/raft

raft: more logging
Xiang Li 11 years ago
parent
commit
a426b310fc
1 changed files with 9 additions and 0 deletions
  1. 9 0
      raft/raft.go

+ 9 - 0
raft/raft.go

@@ -399,11 +399,14 @@ func (r *raft) campaign() {
 	r.becomeCandidate()
 	r.becomeCandidate()
 	if r.q() == r.poll(r.id, true) {
 	if r.q() == r.poll(r.id, true) {
 		r.becomeLeader()
 		r.becomeLeader()
+		return
 	}
 	}
 	for i := range r.prs {
 	for i := range r.prs {
 		if i == r.id {
 		if i == r.id {
 			continue
 			continue
 		}
 		}
+		log.Printf("raft: %x [logterm: %d, index: %d] sent vote request to %x at term %d",
+			r.id, r.raftLog.lastTerm(), r.raftLog.lastIndex(), i, r.Term)
 		r.send(pb.Message{To: i, Type: pb.MsgVote, Index: r.raftLog.lastIndex(), LogTerm: r.raftLog.lastTerm()})
 		r.send(pb.Message{To: i, Type: pb.MsgVote, Index: r.raftLog.lastIndex(), LogTerm: r.raftLog.lastTerm()})
 	}
 	}
 }
 }
@@ -413,7 +416,9 @@ func (r *raft) Step(m pb.Message) error {
 	defer func() { r.Commit = r.raftLog.committed }()
 	defer func() { r.Commit = r.raftLog.committed }()
 
 
 	if m.Type == pb.MsgHup {
 	if m.Type == pb.MsgHup {
+		log.Printf("raft: %x is starting a new election at term %d", r.id, r.Term)
 		r.campaign()
 		r.campaign()
+		return nil
 	}
 	}
 
 
 	switch {
 	switch {
@@ -424,9 +429,13 @@ func (r *raft) Step(m pb.Message) error {
 		if m.Type == pb.MsgVote {
 		if m.Type == pb.MsgVote {
 			lead = None
 			lead = None
 		}
 		}
+		log.Printf("raft: %x [term: %d] received a %s message with higher term from %x [term: %d]",
+			r.id, r.Term, m.Type, m.From, m.Term)
 		r.becomeFollower(m.Term, lead)
 		r.becomeFollower(m.Term, lead)
 	case m.Term < r.Term:
 	case m.Term < r.Term:
 		// ignore
 		// ignore
+		log.Printf("raft: %x [term: %d] ignored a %s message with lower term from %x [term: %d]",
+			r.id, r.Term, m.Type, m.From, m.Term)
 		return nil
 		return nil
 	}
 	}
 	r.step(r, m)
 	r.step(r, m)