kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From a...@apache.org
Subject [kudu] branch master updated: [rebalance] remove unused function declaration and fix typo error in comment.
Date Tue, 04 Jun 2019 07:57:29 GMT
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
     new c45002d  [rebalance] remove unused function declaration and fix typo error in comment.
c45002d is described below

commit c45002dfe4fcba5715ab2763820c9730ec8030ad
Author: zhangyifan27 <chinazhangyifan@163.com>
AuthorDate: Tue Jun 4 14:55:18 2019 +0800

    [rebalance] remove unused function declaration and fix typo error in comment.
    
    GetNextMoves(Runner* runner,std::vector<ReplicaMove>* replica_moves) was
    a function used in previous versions, it seems useless now.
    
    Change-Id: Id3eac211f80d7ec36897488b4f56fe9ed92072ff
    Reviewed-on: http://gerrit.cloudera.org:8080/13505
    Reviewed-by: Adar Dembo <adar@cloudera.com>
    Tested-by: Kudu Jenkins
---
 src/kudu/tools/rebalance_algo.h | 2 +-
 src/kudu/tools/rebalancer.h     | 3 ---
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/kudu/tools/rebalance_algo.h b/src/kudu/tools/rebalance_algo.h
index 7ed1822..22da043 100644
--- a/src/kudu/tools/rebalance_algo.h
+++ b/src/kudu/tools/rebalance_algo.h
@@ -226,7 +226,7 @@ class TwoDimensionalGreedyAlgo : public RebalancingAlgo {
 //
 // ******* Some Examples *******
 //
-// Tablet T of replication factor 5, and locations L_0, ..., L_4. Consider
+// Table T of replication factor 5, and locations L_0, ..., L_4. Consider
 // the following tablet servers disposition:
 //
 //   ts_num(L_0): 2
diff --git a/src/kudu/tools/rebalancer.h b/src/kudu/tools/rebalancer.h
index 9aa3241..cb7fd11 100644
--- a/src/kudu/tools/rebalancer.h
+++ b/src/kudu/tools/rebalancer.h
@@ -482,9 +482,6 @@ class Rebalancer {
   Status GetClusterRawInfo(const boost::optional<std::string>& location,
                            ClusterRawInfo* raw_info);
 
-  Status GetNextMoves(Runner* runner,
-                      std::vector<ReplicaMove>* replica_moves);
-
   // Reset ksck-related fields and run ksck against the cluster.
   Status RefreshKsckResults();
 


Mime
View raw message