Todd Lipcon has submitted this change and it was merged. Change subject: consensus: Don't replay config changes ......................................................................
consensus: Don't replay config changes We have invariants in place that make it unnecessary to "replay" a config change operation, so we will now simply check those invariants during tablet bootstrap. This patch removes plumbing of ConsensusMetadataManager into TabletBootstrap, since it is not needed there, and instead simply passes in the previously-committed raft config from the consensus metadata in order to check for correctness. Change-Id: Id62bf96b21560b2ef5838415e52aeb3720875e62 Reviewed-on: http://gerrit.cloudera.org:8080/7614 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <t...@apache.org> --- M src/kudu/consensus/consensus_meta_manager.cc M src/kudu/master/sys_catalog.cc M src/kudu/tablet/tablet_bootstrap-test.cc M src/kudu/tablet/tablet_bootstrap.cc M src/kudu/tablet/tablet_bootstrap.h M src/kudu/tserver/ts_tablet_manager.cc 6 files changed, 55 insertions(+), 60 deletions(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/7614 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id62bf96b21560b2ef5838415e52aeb3720875e62 Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>