Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14110 )
Change subject: Create kudu/rebalance subdirectory ...................................................................... Create kudu/rebalance subdirectory Moved and renamed structs used for rebalancing from tools/ksck_results into rebalance/cluster_status. Moved tools/rebalance_algo to rebalance/rebalance_algo. Moved class Rebalancer from tools/rebalancer to rebalance/rebalancer. Renamed tools/rebalancer to tools/rebalancer_tool. Created RebalancerTool in tools/rebalancer_tool that inherits from Rebalancer. Moved corresponding tests into kudu/rebalance subdirectory. Movement of code done to prepare for autorebalancer task in master. No functional changes made. Change-Id: Ie0242a019cb44517539da2878cf889ee0c511964 Reviewed-on: http://gerrit.cloudera.org:8080/14110 Tested-by: Andrew Wong <[email protected]> Reviewed-by: Andrew Wong <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M CMakeLists.txt A src/kudu/rebalance/CMakeLists.txt A src/kudu/rebalance/cluster_status.cc A src/kudu/rebalance/cluster_status.h R src/kudu/rebalance/placement_policy_util-test.cc R src/kudu/rebalance/placement_policy_util.cc R src/kudu/rebalance/placement_policy_util.h R src/kudu/rebalance/rebalance-test.cc R src/kudu/rebalance/rebalance_algo-test.cc R src/kudu/rebalance/rebalance_algo.cc R src/kudu/rebalance/rebalance_algo.h A src/kudu/rebalance/rebalancer.cc A src/kudu/rebalance/rebalancer.h M src/kudu/tools/CMakeLists.txt M src/kudu/tools/ksck-test.cc M src/kudu/tools/ksck.cc M src/kudu/tools/ksck.h M src/kudu/tools/ksck_remote.cc M src/kudu/tools/ksck_remote.h M src/kudu/tools/ksck_results.cc M src/kudu/tools/ksck_results.h D src/kudu/tools/rebalancer.h R src/kudu/tools/rebalancer_tool.cc A src/kudu/tools/rebalancer_tool.h M src/kudu/tools/tool.proto M src/kudu/tools/tool_action_cluster.cc 26 files changed, 1,918 insertions(+), 1,627 deletions(-) Approvals: Andrew Wong: Looks good to me, but someone else must approve; Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14110 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie0242a019cb44517539da2878cf889ee0c511964 Gerrit-Change-Number: 14110 Gerrit-PatchSet: 14 Gerrit-Owner: Hannah Nguyen <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hannah Nguyen <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
