Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10819 )
Change subject: [tools] small fix on RebalancingAlgo::GetNextMoves() ...................................................................... [tools] small fix on RebalancingAlgo::GetNextMoves() Fix on short-circuiting in case of empty input for RebalancingAlgo::GetNextMoves(). Added consistency check for the 'cluster_info' input parameter and corresponding unit test. Change-Id: Iad7a7bc5bee939ce92d18845d6e43bcadccac5a2 Reviewed-on: http://gerrit.cloudera.org:8080/10819 Reviewed-by: Mike Percy <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/tools/rebalance_algo-test.cc M src/kudu/tools/rebalance_algo.cc 2 files changed, 36 insertions(+), 4 deletions(-) Approvals: Mike Percy: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/10819 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iad7a7bc5bee939ce92d18845d6e43bcadccac5a2 Gerrit-Change-Number: 10819 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]>
