You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by to...@apache.org on 2016/09/20 03:51:31 UTC

[2/3] kudu git commit: consensus: remove some unimplemented methods

consensus: remove some unimplemented methods

Change-Id: I82089fa5a4d1e62e63c2656cc6ffeefccf10376c
Reviewed-on: http://gerrit.cloudera.org:8080/4452
Reviewed-by: David Ribeiro Alves <dr...@apache.org>
Tested-by: Kudu Jenkins


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/c9b1fefd
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/c9b1fefd
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/c9b1fefd

Branch: refs/heads/master
Commit: c9b1fefd25162ae79fac76d1670bc6f08234d139
Parents: 9aab93f
Author: Todd Lipcon <to...@apache.org>
Authored: Mon Sep 19 14:44:46 2016 -0700
Committer: Todd Lipcon <to...@apache.org>
Committed: Tue Sep 20 02:01:11 2016 +0000

----------------------------------------------------------------------
 src/kudu/consensus/raft_consensus_state.h | 12 ------------
 1 file changed, 12 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/c9b1fefd/src/kudu/consensus/raft_consensus_state.h
----------------------------------------------------------------------
diff --git a/src/kudu/consensus/raft_consensus_state.h b/src/kudu/consensus/raft_consensus_state.h
index f73cccd..174458f 100644
--- a/src/kudu/consensus/raft_consensus_state.h
+++ b/src/kudu/consensus/raft_consensus_state.h
@@ -287,18 +287,6 @@ class ReplicaState {
   // latest index). This must be called under the lock.
   OpId GetLastPendingTransactionOpIdUnlocked() const;
 
-  // Updates the last committed operation including removing it from the pending commits.
-  //
-  // 'commit_op_id' refers to the OpId of the actual commit operation, whereas
-  // 'committed_op_id' refers to the OpId of the original REPLICATE message which was
-  // committed.
-  //
-  // This must be called under a lock.
-  void UpdateReplicaCommittedOpIdUnlocked(const OpId& committed_op_id);
-
-  // Waits for already triggered Apply()s to commit.
-  Status WaitForOustandingApplies();
-
   // Used by replicas to cancel pending transactions. Pending transaction are those
   // that have completed prepare/replicate but are waiting on the LEADER's commit
   // to complete. This does not cancel transactions being applied.