Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15850
Change subject: KUDU-3113: fix auto-rebalancer move execution ...................................................................... KUDU-3113: fix auto-rebalancer move execution When executing moves, the auto-rebalancer would try to resolve the leader's address by passing its UUID instead of its host. This fixes it to use an appropriate host. This includes some light cleanup, and updates auto_rebalancer-test to verify the moves lead to the copying of bytes on tablet servers. I also validated this on a real cluster: - First, I enabled auto-rebalancing on the master. - I put a tablet server into mainatenance mode. - I then moved all replicas off the tablet server using the rebalancer tool's --move_replicas_from_ignored_tservers option. - I verified that even with the great skew, since one of the tablet servers was in maintenance mode, the master didn't automatically move any replicas. - Once I took the tablet server out of maintenance mode, moves were scheduled to repopulate it. - Steady state was reached with a cluster skew of 1. Change-Id: If658997dc9bcb709c27d981db56cf2db13ba235f --- M src/kudu/master/auto_rebalancer-test.cc M src/kudu/master/auto_rebalancer.cc 2 files changed, 167 insertions(+), 98 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/50/15850/1 -- To view, visit http://gerrit.cloudera.org:8080/15850 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: If658997dc9bcb709c27d981db56cf2db13ba235f Gerrit-Change-Number: 15850 Gerrit-PatchSet: 1 Gerrit-Owner: Andrew Wong <[email protected]>
