Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10336 )
Change subject: [rebalancing] add a rebalancing algorithm ...................................................................... Patch Set 19: (2 comments) http://gerrit.cloudera.org:8080/#/c/10336/19/src/kudu/tools/rebalance_algo.cc File src/kudu/tools/rebalance_algo.cc: http://gerrit.cloudera.org:8080/#/c/10336/19/src/kudu/tools/rebalance_algo.cc@88 PS19, Line 88: DCHECK(found_src && found_dst); > since you're handling this case now, maybe remove this? Done http://gerrit.cloudera.org:8080/#/c/10336/19/src/kudu/tools/rebalance_algo.cc@117 PS19, Line 117: Value of '0' is a shortcut for 'the possible maximum'. > do we need this? if so, please doc this It's a handy shortcut to use in tests, etc. I documented that in the header file rebalance_algo.h -- To view, visit http://gerrit.cloudera.org:8080/10336 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5a8050ee79117a2ae2f6f88740ed25656946cfb4 Gerrit-Change-Number: 10336 Gerrit-PatchSet: 19 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Thu, 07 Jun 2018 03:05:31 +0000 Gerrit-HasComments: Yes
