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

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


Patch Set 3:

(1 comment)

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), 
consider also adding a test for 'kudu cluster rebalance --help' output if you 
are have extra time to work on this.  As an example, take a look at 
http://gerrit.cloudera.org:8080/23555



--
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: 3
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, 03 Aug 2026 18:53:07 +0000
Gerrit-HasComments: Yes

Reply via email to