tursom/memgraph
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Prevent snapshot recovery when exiting

Browse Source 
Summary:
Sometimes, when the leader resigns it's leadership, a newly elected
leader would send the old leader `AppendEntriesRPC` that would cause the
snapshot recovery to happen. This diff prevents that.

Reviewers: mferencevic, ipaljak

Reviewed By: mferencevic

Subscribers: pullbot

Differential Revision: https://phabricator.memgraph.io/D1943
This commit is contained in:
Matija Santl 2019-04-04 13:06:05 +02:00
parent 6a9acb717d
commit e1ad5cd803
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/raft/raft_server.cpp
View File

@ -1,6 +1,7 @@
#include "raft/raft_server.hpp"
#include <kj/std/iostream.h>
#include <algorithm>
#include <chrono>
#include <iostream>
#include <memory>
@ -128,7 +129,7 @@ void RaftServer::Start() {
// [Raft paper 5.1]
// "If a server receives a request with a stale term, it rejects the
// request"
if (req.term < current_term_) {
if (exiting_ || req.term < current_term_) {
AppendEntriesRes res(false, current_term_);
Save(res, res_builder);
return;
@ -744,9 +745,9 @@ void RaftServer::SendLogEntries(
}
if (!reply.success) {
DCHECK(next_index_[peer_id] > 1)
<< "Log replication should not fail for first log entry";
--next_index_[peer_id];
// Replication can fail for the first log entry if the peer that we're
// sending the entry is in the process of shutting down.
next_index_[peer_id] = std::max(next_index_[peer_id] - 1, 1UL);
} else {
uint64_t new_match_index = request_prev_log_index + request_entries.size();
DCHECK(match_index_[peer_id] <= new_match_index)