Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8124
Change subject: [consensus] don't crash in RaftConsensus::ChangeConfig ...................................................................... [consensus] don't crash in RaftConsensus::ChangeConfig Don't crash if ReplicateConfigChangeUnlocked() returns an error. Instead, just return corresponding error to the caller. This is to avoid crashing a tablet server if an invalid Raft configuration is passed to ChangeConfig() RPC with ChangeConfigRequestPB parameter. Also, use move semantics for all parameters of the RaftConsensus::ReplicateConfigChangeUnlocked() method. Change-Id: Ieac3692adae029d098fea254badf2c53c6ac4fd8 --- M src/kudu/consensus/raft_consensus.cc M src/kudu/consensus/raft_consensus.h 2 files changed, 11 insertions(+), 13 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/24/8124/1 -- To view, visit http://gerrit.cloudera.org:8080/8124 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ieac3692adae029d098fea254badf2c53c6ac4fd8 Gerrit-Change-Number: 8124 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
