You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2019/10/11 06:15:21 UTC

[kudu] 01/03: [tablet] remove used method in Mutation

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git

commit b6e4672092292acf7316a700c71fad3bb54ffa77
Author: lingbin <li...@gmail.com>
AuthorDate: Thu Oct 10 22:28:36 2019 +0800

    [tablet] remove used method in Mutation
    
    Change-Id: Ic3c7cded06ce7fead5f56371166e3908d16efa7a
    Reviewed-on: http://gerrit.cloudera.org:8080/14409
    Tested-by: Kudu Jenkins
    Reviewed-by: Alexey Serbin <as...@cloudera.com>
---
 src/kudu/tablet/mutation.h | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/kudu/tablet/mutation.h b/src/kudu/tablet/mutation.h
index 9071665..5a28c13 100644
--- a/src/kudu/tablet/mutation.h
+++ b/src/kudu/tablet/mutation.h
@@ -94,17 +94,13 @@ class Mutation {
     *list = this;
   }
 
-  // O(n) algorithm to reverse the order of a linked list of
-  // mutations.
+  // O(n) algorithm to reverse the order of a linked list of mutations.
   static void ReverseMutationList(Mutation** list);
 
  private:
   friend class MSRow;
   friend class MemRowSet;
 
-  template<bool ATOMIC>
-  void DoAppendToList(Mutation **list);
-
   // The transaction ID which made this mutation. If this transaction is not
   // committed in the snapshot of the reader, this mutation should be ignored.
   Timestamp timestamp_;