You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Mike Percy (Code Review)" <ge...@cloudera.org> on 2017/05/27 20:00:07 UTC

[kudu-CR] consensus: Get rid of ReplicaState class

Hello Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/7007

to look at the new patch set (#2).

Change subject: consensus: Get rid of ReplicaState class
......................................................................

consensus: Get rid of ReplicaState class

Merges the logic in ReplicaState into the RaftConsensus class.
ReplicaState adds complexity but doesn't really serve a purpose anymore.

There are no functional changes in this patch.

Change-Id: Ie1e62eff37d3f8655100b364939375608063aa80
---
M src/kudu/consensus/CMakeLists.txt
M src/kudu/consensus/consensus.h
M src/kudu/consensus/raft_consensus.cc
M src/kudu/consensus/raft_consensus.h
M src/kudu/consensus/raft_consensus_quorum-test.cc
D src/kudu/consensus/raft_consensus_state.cc
D src/kudu/consensus/raft_consensus_state.h
7 files changed, 697 insertions(+), 871 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/07/7007/2
-- 
To view, visit http://gerrit.cloudera.org:8080/7007
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie1e62eff37d3f8655100b364939375608063aa80
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Tidy Bot