Yifan Zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/18074 )
Change subject: KUDU-3328: make rebalancer tool take into account maintenance mode ...................................................................... Patch Set 4: (5 comments) http://gerrit.cloudera.org:8080/#/c/18074/3//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/18074/3//COMMIT_MSG@15 PS3, Line 15: not specified in 'ignored_tservers' and users don't specify > It would be great to add small blurb to explain the target use case for thi Done http://gerrit.cloudera.org:8080/#/c/18074/3/src/kudu/tools/rebalancer_tool.h File src/kudu/tools/rebalancer_tool.h: http://gerrit.cloudera.org:8080/#/c/18074/3/src/kudu/tools/rebalancer_tool.h@100 PS3, Line 100: tablets server > nit: tablet servers Done http://gerrit.cloudera.org:8080/#/c/18074/3/src/kudu/tools/rebalancer_tool.cc File src/kudu/tools/rebalancer_tool.cc: http://gerrit.cloudera.org:8080/#/c/18074/3/src/kudu/tools/rebalancer_tool.cc@763 PS3, Line 763: tserve > nit: maybe, use 'tserver' here as well since it's referred to as 'tserver' Done http://gerrit.cloudera.org:8080/#/c/18074/3/src/kudu/tools/tool_action_cluster.cc File src/kudu/tools/tool_action_cluster.cc: http://gerrit.cloudera.org:8080/#/c/18074/3/src/kudu/tools/tool_action_cluster.cc@156 PS3, Line 156: force_rebalance_replicas_on_maintenance_tservers > It would be nice to clarify on the behavior of the tool if the following ar In terms of current implementation the sematic of 'ignored_tservers' is stronger than this 'force_...' flag, replicas on the tserver and the tserver itself would be effectively ignored as long as it is specified in 'ignored_tservers'. http://gerrit.cloudera.org:8080/#/c/18074/3/src/kudu/tools/tool_action_cluster.cc@159 PS3, Line 159: tservers. The si > Any adverse effects when doing so? For example, what happens if a tablet s Done -- To view, visit http://gerrit.cloudera.org:8080/18074 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I843a2521a811ab67c5f98bb43cc84367edb3fe0b Gerrit-Change-Number: 18074 Gerrit-PatchSet: 4 Gerrit-Owner: Yifan Zhang <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Comment-Date: Tue, 21 Dec 2021 07:55:50 +0000 Gerrit-HasComments: Yes
