You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2022/03/10 23:01:24 UTC

[kudu-CR] [tools] run intra-location rebalancing in parallel

Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/18308


Change subject: [tools] run intra-location rebalancing in parallel
......................................................................

[tools] run intra-location rebalancing in parallel

This patch addresses TODO in RebalancerTool::Run() to run intra-location
rebalancing in parallel for location-aware Kudu cluster.  Since the
location assignment provides an invariant for a tablet server to be only
in one location, a locatation-aware Kudu cluster automatically consists
of non-intersecting set of tablet servers.  With that, it's possible to
independently move replicas within different locations while satisfying
various run-time properties, e.g. set by --max_moves_per_server and
other flags.

Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
---
M src/kudu/rebalance/rebalancer.cc
M src/kudu/rebalance/rebalancer.h
M src/kudu/tools/rebalancer_tool.cc
M src/kudu/tools/rebalancer_tool.h
M src/kudu/tools/tool_action_cluster.cc
5 files changed, 161 insertions(+), 28 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/08/18308/1
-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18308 )

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................

[tools] run intra-location rebalancing in parallel

This patch addresses TODO in RebalancerTool::Run() to run intra-location
rebalancing in parallel for location-aware Kudu cluster.  Since the
location assignment provides an invariant for a tablet server to be only
in one location, a locatation-aware Kudu cluster automatically consists
of non-intersecting set of tablet servers.  With that, it's possible to
independently move replicas within different locations while satisfying
various run-time properties, e.g. set by --max_moves_per_server and
other flags.

The new binary was tested on a large production cluster where location
awareness was enabled and it did significantly speed up the rebalancing
process.  Running the binary in my test cluster showed significant
reduction of run times as well.  Theoretically, running intra-location
rebalancing in parallel might shorten the runtime by N times compared
with running sequentially, where N is the number of locations
in a Kudu cluster.

Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Reviewed-on: http://gerrit.cloudera.org:8080/18308
Tested-by: Alexey Serbin <al...@apache.org>
Reviewed-by: Alexey Serbin <al...@apache.org>
---
M src/kudu/rebalance/rebalancer.cc
M src/kudu/rebalance/rebalancer.h
M src/kudu/tools/rebalancer_tool-test.cc
M src/kudu/tools/rebalancer_tool.cc
M src/kudu/tools/rebalancer_tool.h
M src/kudu/tools/tool_action_cluster.cc
6 files changed, 191 insertions(+), 28 deletions(-)

Approvals:
  Alexey Serbin: Looks good to me, approved; Verified

-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18308 )

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................


Patch Set 3: Code-Review+2

Carrying over Andrew's +2 from PS2


-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 17 Mar 2022 23:03:02 +0000
Gerrit-HasComments: No

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has removed a vote on this change.

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................


Removed Verified-1 by Kudu Jenkins (120)
-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18308 )

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/18308/1/src/kudu/tools/rebalancer_tool.cc
File src/kudu/tools/rebalancer_tool.cc:

http://gerrit.cloudera.org:8080/#/c/18308/1/src/kudu/tools/rebalancer_tool.cc@324
PS1, Line 324: s.message()
> nit: do you think it's worth adding information about which location had wh
Yes, I think that's a good idea.

Done.


http://gerrit.cloudera.org:8080/#/c/18308/1/src/kudu/tools/rebalancer_tool.cc@328
PS1, Line 328:       if (!status.ok()) {
             :         return status;
             :       }
> nit: RETURN_NOT_OK(status)?
Done


http://gerrit.cloudera.org:8080/#/c/18308/1/src/kudu/tools/rebalancer_tool.cc@737
PS1, Line 737: guard
> nit: maybe name this refresh_lock so it's less error prone (e.g. easy to co
renamed into 'refresh_guard'



-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 14 Mar 2022 23:57:42 +0000
Gerrit-HasComments: Yes

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18308 )

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................


Patch Set 3:

Carrying over Andrew's +2 from PS2


-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 17 Mar 2022 23:02:45 +0000
Gerrit-HasComments: No

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Abhishek Chennaka, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/18308

to look at the new patch set (#3).

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................

[tools] run intra-location rebalancing in parallel

This patch addresses TODO in RebalancerTool::Run() to run intra-location
rebalancing in parallel for location-aware Kudu cluster.  Since the
location assignment provides an invariant for a tablet server to be only
in one location, a locatation-aware Kudu cluster automatically consists
of non-intersecting set of tablet servers.  With that, it's possible to
independently move replicas within different locations while satisfying
various run-time properties, e.g. set by --max_moves_per_server and
other flags.

The new binary was tested on a large production cluster where location
awareness was enabled and it did significantly speed up the rebalancing
process.  Running the binary in my test cluster showed significant
reduction of run times as well.  Theoretically, running intra-location
rebalancing in parallel might shorten the runtime by N times compared
with running sequentially, where N is the number of locations
in a Kudu cluster.

Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
---
M src/kudu/rebalance/rebalancer.cc
M src/kudu/rebalance/rebalancer.h
M src/kudu/tools/rebalancer_tool-test.cc
M src/kudu/tools/rebalancer_tool.cc
M src/kudu/tools/rebalancer_tool.h
M src/kudu/tools/tool_action_cluster.cc
6 files changed, 191 insertions(+), 28 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/08/18308/3
-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has removed a vote on this change.

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................


Removed Verified-1 by Kudu Jenkins (120)
-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Andrew Wong (Code Review)" <ge...@cloudera.org>.
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/18308 )

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................


Patch Set 2: Code-Review+2

LGTM. For posterity, it's also worth noting that this was tested on a large production cluster and it did significantly speed up their rebalancing.


-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 17 Mar 2022 20:26:14 +0000
Gerrit-HasComments: No

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18308 )

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................


Patch Set 1: Verified+1

unrelated test failures in:
  * ParameterizedRecoverMasterTest.TestRecoverDeadMasterSysCatalogCopy/0
  * RaftConsensusITest.TestSlowFollower


-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 11 Mar 2022 00:09:53 +0000
Gerrit-HasComments: No

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18308 )

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................


Patch Set 2: Verified+1

unrelated test failures (DEBUG configuration):
  * ParameterizedRecoverMasterTest.TestRecoverDeadMasterSysCatalogCopy/1
  * DisableWriteWhenExceedingQuotaTest.TestDisableWritePrivilegeWhenExceedingSizeQuota


-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 16 Mar 2022 21:21:49 +0000
Gerrit-HasComments: No

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/18308 )

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................


Patch Set 3: Verified+1

unrelated test failure in RaftConsensusITest.TestSlowFollower


-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 17 Mar 2022 23:02:23 +0000
Gerrit-HasComments: No

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has removed a vote on this change.

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................


Removed Verified-1 by Kudu Jenkins (120)
-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Andrew Wong (Code Review)" <ge...@cloudera.org>.
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/18308 )

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................


Patch Set 1: Code-Review+1

(3 comments)

http://gerrit.cloudera.org:8080/#/c/18308/1/src/kudu/tools/rebalancer_tool.cc
File src/kudu/tools/rebalancer_tool.cc:

http://gerrit.cloudera.org:8080/#/c/18308/1/src/kudu/tools/rebalancer_tool.cc@324
PS1, Line 324: s.message()
nit: do you think it's worth adding information about which location had which status? Is that easy to figure out from the logs?


http://gerrit.cloudera.org:8080/#/c/18308/1/src/kudu/tools/rebalancer_tool.cc@328
PS1, Line 328:       if (!status.ok()) {
             :         return status;
             :       }
nit: RETURN_NOT_OK(status)?


http://gerrit.cloudera.org:8080/#/c/18308/1/src/kudu/tools/rebalancer_tool.cc@737
PS1, Line 737: guard
nit: maybe name this refresh_lock so it's less error prone (e.g. easy to conflate with the held ksck_lock below



-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 14 Mar 2022 22:13:45 +0000
Gerrit-HasComments: Yes

[kudu-CR] [tools] run intra-location rebalancing in parallel

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Hello Attila Bukor, Kudu Jenkins, Andrew Wong, Abhishek Chennaka, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/18308

to look at the new patch set (#2).

Change subject: [tools] run intra-location rebalancing in parallel
......................................................................

[tools] run intra-location rebalancing in parallel

This patch addresses TODO in RebalancerTool::Run() to run intra-location
rebalancing in parallel for location-aware Kudu cluster.  Since the
location assignment provides an invariant for a tablet server to be only
in one location, a locatation-aware Kudu cluster automatically consists
of non-intersecting set of tablet servers.  With that, it's possible to
independently move replicas within different locations while satisfying
various run-time properties, e.g. set by --max_moves_per_server and
other flags.

Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
---
M src/kudu/rebalance/rebalancer.cc
M src/kudu/rebalance/rebalancer.h
M src/kudu/tools/rebalancer_tool-test.cc
M src/kudu/tools/rebalancer_tool.cc
M src/kudu/tools/rebalancer_tool.h
M src/kudu/tools/tool_action_cluster.cc
6 files changed, 191 insertions(+), 28 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/08/18308/2
-- 
To view, visit http://gerrit.cloudera.org:8080/18308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie4fe3ef3ec2fcac57114c97d5b6cd81d5d9953c4
Gerrit-Change-Number: 18308
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)