Merge pull request #315 from mmaslankaprv/recovery-state-update-fix

r/consensus: fixed updating follower state when its data were deleted
redpanda-data · Dec 21, 2020 · be35da8 · be35da8
2 parents ec8fd30 + cb6db32
commit be35da8
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/v/raft/consensus.cc b/src/v/raft/consensus.cc
@@ -266,6 +266,10 @@ consensus::success_reply consensus::update_follower_index(
  // we are already recovering, if follower dirty log index moved back
  // from some reason (i.e. truncation, data loss, trigger recovery)
  if (idx.last_dirty_log_index > reply.last_dirty_log_index) {
+ // update follower state to allow recovery of follower with
+ // missing entries
+ idx.last_dirty_log_index = reply.last_dirty_log_index;
+ idx.last_committed_log_index = reply.last_committed_log_index;
  idx.next_index = details::next_offset(idx.last_dirty_log_index);
  idx.follower_state_change.broadcast();
  }