Mike Percy has submitted this change and it was merged.

Change subject: Use RaftConsensus instead of LocalConsensus in tests
......................................................................


Use RaftConsensus instead of LocalConsensus in tests

This paves the way to remove the LocalConsensus implementation.

Also add WaitUntilLeader() to Consensus interface for use by tests.

Changes the following tests to use RaftConsensus:

* tablet_peer-test
* remote_bootstrap_client-test
* remote_bootstrap_session-test
* tablet_server-test
* ts_tablet_manager-test

Change-Id: I72e94d7e78e3428a8d9696737e07b8e8f7489d49
Reviewed-on: http://gerrit.cloudera.org:8080/3346
Tested-by: Mike Percy <mpe...@apache.org>
Reviewed-by: David Ribeiro Alves <dral...@apache.org>
---
M src/kudu/consensus/consensus.h
M src/kudu/consensus/local_consensus.h
M src/kudu/consensus/raft_consensus.cc
M src/kudu/consensus/raft_consensus.h
M src/kudu/tablet/tablet_peer-test.cc
M src/kudu/tserver/mini_tablet_server.cc
M src/kudu/tserver/remote_bootstrap-test-base.h
M src/kudu/tserver/remote_bootstrap_client-test.cc
M src/kudu/tserver/remote_bootstrap_session-test.cc
M src/kudu/tserver/tablet_server-stress-test.cc
M src/kudu/tserver/tablet_server-test-base.h
M src/kudu/tserver/tablet_server-test.cc
M src/kudu/tserver/ts_tablet_manager-test.cc
13 files changed, 68 insertions(+), 39 deletions(-)

Approvals:
  David Ribeiro Alves: Looks good to me, approved
  Mike Percy: Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/3346
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I72e94d7e78e3428a8d9696737e07b8e8f7489d49
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>

Reply via email to