Gabriella Lotz has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24613 )

Change subject: KUDU-3760 expose follower-move flag in rebalancer
......................................................................


Patch Set 4:

(2 comments)

> Patch Set 3:
>
> Thank you! Did you have a chance to take a look at my comment about 
> src/kudu/tools/tool_action_cluster.cc?

Yes, I have updated it!

http://gerrit.cloudera.org:8080/#/c/24613/3/src/kudu/tools/rebalancer_tool-test.cc
File src/kudu/tools/rebalancer_tool-test.cc:

http://gerrit.cloudera.org:8080/#/c/24613/3/src/kudu/tools/rebalancer_tool-test.cc@734
PS3, Line 734: (an unregistered flag would abort the tool)
> To cover the other side (i.e. keeping the registered flags documented), con
Good idea, although it is broader than this change. I filed KUDU-3798 as a 
follow-up. Thanks for the pointer.


http://gerrit.cloudera.org:8080/#/c/24613/3/src/kudu/tools/rebalancer_tool.cc
File src/kudu/tools/rebalancer_tool.cc:

http://gerrit.cloudera.org:8080/#/c/24613/3/src/kudu/tools/rebalancer_tool.cc@106
PS3, Line 106: prefer_follo
> Since this flag is exposed only through 'kudu cluster rebalance' CLI, consi
Done



--
To view, visit http://gerrit.cloudera.org:8080/24613
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5f1b03da71b11af82f18ff9e20dcaa676b185645
Gerrit-Change-Number: 24613
Gerrit-PatchSet: 4
Gerrit-Owner: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Comment-Date: Mon, 10 Aug 2026 14:49:16 +0000
Gerrit-HasComments: Yes

Reply via email to