You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Yifan Zhang (Code Review)" <ge...@cloudera.org> on 2019/10/01 16:18:49 UTC

[kudu-CR] KUDU-2914: Rebalance tool support moving replicas from some specific tablet servers

Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, 

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

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

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

Change subject: KUDU-2914: Rebalance tool support moving replicas from some specific tablet servers
......................................................................

KUDU-2914: Rebalance tool support moving replicas from some specific tablet servers

Aims to support moving replicas from specific tablet servers,
this patch re-uses the '--ignored_tservers' flag and adds a
'--move_replicas_from_ignored_tservers' flag to the
`kudu rebalance cluster` CLI tool.

Once the flag '--ignored_tservers' is specified, and the flag
'--move_replicas_from_ignored_tservers' is specified false,
the given tablet servers are not considered as a part of the
cluster, their health state and replicas on them are ignored
by the rebalancer tool.

If '--move_replicas_from_ignored_tservers' is enabled, unhealthy
tservers in the given ignored tservers will also be ignored by
the rebalancer tool. But for healthy ignored tservers, replicas
on them would be moved to other tservers before starting the
rebalancing process, when running the rebalancing, these tservers
are not considered as a part of the cluster.

Change-Id: I86cfb740030946c13db1a9ca63d241f4907d6c89
---
M src/kudu/rebalance/rebalance-test.cc
M src/kudu/rebalance/rebalance_algo-test.cc
M src/kudu/rebalance/rebalance_algo.cc
M src/kudu/rebalance/rebalance_algo.h
M src/kudu/rebalance/rebalancer.cc
M src/kudu/rebalance/rebalancer.h
M src/kudu/tools/rebalancer_tool-test.cc
M src/kudu/tools/rebalancer_tool.cc
M src/kudu/tools/rebalancer_tool.h
M src/kudu/tools/tool_action_cluster.cc
10 files changed, 865 insertions(+), 68 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/54/14154/5
-- 
To view, visit http://gerrit.cloudera.org:8080/14154
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I86cfb740030946c13db1a9ca63d241f4907d6c89
Gerrit-Change-Number: 14154
Gerrit-PatchSet: 5
Gerrit-Owner: Yifan Zhang <ch...@163.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yifan Zhang <ch...@163.com>