[kudu] branch master updated: [consensus] remove redundant instruction

[alexey]

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 13ce789  [consensus] remove redundant instruction
13ce789 is described below

commit 13ce789016310bb3b76fc21bf907a4e5f25c18f5
Author: Alexey Serbin <ale...@apache.org>
AuthorDate: Thu Jun 18 17:27:21 2020 -0700

    [consensus] remove redundant instruction
    
    Removed assignment to leader_transfer_in_progress_ after calling
    std::atomic_exchange with the same desired value.
    
    Change-Id: I4bbd40d89bcb39581dba15699bbfc9ed4036b11e
    Reviewed-on: http://gerrit.cloudera.org:8080/16097
    Tested-by: Kudu Jenkins
    Reviewed-by: Bankim Bhavsar <ban...@cloudera.com>
---
 src/kudu/consensus/raft_consensus.cc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/kudu/consensus/raft_consensus.cc 
b/src/kudu/consensus/raft_consensus.cc
index b9418e4..cefa1bf 100644
--- a/src/kudu/consensus/raft_consensus.cc
+++ b/src/kudu/consensus/raft_consensus.cc
@@ -628,7 +628,6 @@ Status RaftConsensus::BeginLeaderTransferPeriodUnlocked(
         Substitute("leadership transfer for $0 already in progress",
                    options_.tablet_id));
   }
-  leader_transfer_in_progress_ = true;
   queue_->BeginWatchForSuccessor(successor_uuid);
   transfer_period_timer_->Start();
   return Status::OK();