[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-31 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11748 )

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..


Patch Set 7: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 7
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Thu, 01 Nov 2018 03:32:09 +
Gerrit-HasComments: No


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-31 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/11748 )

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..

[rebalancer] location-aware rebalancer (part 7/n)

Added PolicyFixer and integrated the cross-location balancing
algorithm.  Added one basic integration test as well.

More integration tests will be added in a follow-up commit.

Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Reviewed-on: http://gerrit.cloudera.org:8080/11748
Tested-by: Kudu Jenkins
Reviewed-by: Will Berkeley 
---
M src/kudu/tools/CMakeLists.txt
M src/kudu/tools/rebalancer.cc
M src/kudu/tools/rebalancer.h
M src/kudu/tools/rebalancer_tool-test.cc
M src/kudu/tools/tool_action_cluster.cc
5 files changed, 740 insertions(+), 68 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Will Berkeley: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 8
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-31 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins,

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

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

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

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..

[rebalancer] location-aware rebalancer (part 7/n)

Added PolicyFixer and integrated the cross-location balancing
algorithm.  Added one basic integration test as well.

More integration tests will be added in a follow-up commit.

Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
---
M src/kudu/tools/CMakeLists.txt
M src/kudu/tools/rebalancer.cc
M src/kudu/tools/rebalancer.h
M src/kudu/tools/rebalancer_tool-test.cc
M src/kudu/tools/tool_action_cluster.cc
5 files changed, 740 insertions(+), 68 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/11748/7
--
To view, visit http://gerrit.cloudera.org:8080/11748
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 7
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-31 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11748 )

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..


Patch Set 6: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 6
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Thu, 01 Nov 2018 02:05:24 +
Gerrit-HasComments: No


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-31 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins,

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

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

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

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..

[rebalancer] location-aware rebalancer (part 7/n)

Added PolicyFixer and integrated the cross-location balancing
algorithm.  Added one basic integration test as well.

More integration tests will be added in a follow-up commit.

Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
---
M src/kudu/tools/rebalancer.cc
M src/kudu/tools/rebalancer.h
M src/kudu/tools/rebalancer_tool-test.cc
M src/kudu/tools/tool_action_cluster.cc
4 files changed, 738 insertions(+), 67 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/11748/6
--
To view, visit http://gerrit.cloudera.org:8080/11748
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 6
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-31 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins,

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

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

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

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..

[rebalancer] location-aware rebalancer (part 7/n)

Added PolicyFixer and integrated the cross-location balancing
algorithm.  Added one integration test as well.

More integration tests will be added in a follow-up commit.

Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
---
M src/kudu/tools/rebalancer.cc
M src/kudu/tools/rebalancer.h
M src/kudu/tools/rebalancer_tool-test.cc
M src/kudu/tools/tool_action_cluster.cc
4 files changed, 714 insertions(+), 49 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/11748/5
--
To view, visit http://gerrit.cloudera.org:8080/11748
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 5
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-29 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11748 )

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..


Patch Set 4: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Mon, 29 Oct 2018 23:17:46 +
Gerrit-HasComments: No


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11748 )

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11748/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/11748/2//COMMIT_MSG@12
PS2, Line 12: a
> Extra "a".
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Mon, 29 Oct 2018 20:55:02 +
Gerrit-HasComments: Yes


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-29 Thread Alexey Serbin (Code Review)
Hello Will Berkeley,

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

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

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

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..

[rebalancer] location-aware rebalancer (part 7/n)

Added PolicyFixer and integrated the cross-location balancing
algorithm.

Added one integration test as well, but it's disabled for now since
it requires a tablet server's location to be reported in
KsckServerHealthSummary.  If that piece is in place, the test passes.

More integration tests will be added in a follow-up commit.

Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
---
M src/kudu/tools/rebalancer.cc
M src/kudu/tools/rebalancer.h
M src/kudu/tools/rebalancer_tool-test.cc
M src/kudu/tools/tool_action_cluster.cc
4 files changed, 709 insertions(+), 49 deletions(-)


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Will Berkeley 


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-29 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11748 )

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..


Patch Set 2: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11748/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/11748/2//COMMIT_MSG@12
PS2, Line 12: a
Extra "a".



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Mon, 29 Oct 2018 17:34:43 +
Gerrit-HasComments: Yes


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11748 )

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..


Patch Set 2: Verified+1

Unrelated flake in AdminCliTest.TestSimultaneousLeaderTransferAndAbruptStepdown 
(ASAN).


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Tue, 23 Oct 2018 19:36:09 +
Gerrit-HasComments: No


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change.  ( 
http://gerrit.cloudera.org:8080/11748 )

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..


Removed reviewer Kudu Jenkins with the following votes:

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: deleteReviewer
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Will Berkeley 


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11748 )

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..


Patch Set 1:

(5 comments)

http://gerrit.cloudera.org:8080/#/c/11748/1/src/kudu/tools/rebalancer.h
File src/kudu/tools/rebalancer.h:

http://gerrit.cloudera.org:8080/#/c/11748/1/src/kudu/tools/rebalancer.h@326
PS1, Line 326: If 'location' is boost::none, rebalance across
 : // locations.
> The idea is that this balances within a location if 'location' is not boost
Woops, that's just a stale comment.  I updated it.


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

http://gerrit.cloudera.org:8080/#/c/11748/1/src/kudu/tools/rebalancer.cc@257
PS1, Line 257: 
> in
Done


http://gerrit.cloudera.org:8080/#/c/11748/1/src/kudu/tools/rebalancer.cc@289
PS1, Line 289: // TODO(aserbin): it would be nice to run these rebalancers in 
parallel
> Yeah, I think that's a good idea. But also worth saving for a followup-once
Ack.


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

http://gerrit.cloudera.org:8080/#/c/11748/1/src/kudu/tools/rebalancer_tool-test.cc@1170
PS1, Line 1170: This tests
> These tests are
Done


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

http://gerrit.cloudera.org:8080/#/c/11748/1/src/kudu/tools/tool_action_cluster.cc@107
PS1, Line 107: envolves
> involves
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Tue, 23 Oct 2018 19:05:28 +
Gerrit-HasComments: Yes


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-23 Thread Alexey Serbin (Code Review)
Hello Will Berkeley,

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

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

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

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..

[rebalancer] location-aware rebalancer (part 7/n)

Added PolicyFixer and integrated the cross-location balancing
algorithm.

Added one integration test as well, but it's disabled for a now since
it requires a tablet server's location to be reported in
KsckServerHealthSummary.  If that piece is in place, the test passes.

More integration tests will be added in a follow-up commit.

Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
---
M src/kudu/tools/rebalancer.cc
M src/kudu/tools/rebalancer.h
M src/kudu/tools/rebalancer_tool-test.cc
M src/kudu/tools/tool_action_cluster.cc
4 files changed, 709 insertions(+), 49 deletions(-)


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Will Berkeley 


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-22 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11748 )

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..


Patch Set 1:

(5 comments)

http://gerrit.cloudera.org:8080/#/c/11748/1/src/kudu/tools/rebalancer.h
File src/kudu/tools/rebalancer.h:

http://gerrit.cloudera.org:8080/#/c/11748/1/src/kudu/tools/rebalancer.h@326
PS1, Line 326: If 'location' is boost::none, rebalance across
 : // locations.
The idea is that this balances within a location if 'location' is not 
boost::none, and it balances without regard for location otherwise? And then 
this runner can cover both the location-aware and no-locations-assigned cases? 
If so, I think we should say that explicitly.  In fact, do we want it to be 
valid to run without regard for location against a cluster with locations set? 
Should we detect that when the runner runs and fail with an appropriate error?


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

http://gerrit.cloudera.org:8080/#/c/11748/1/src/kudu/tools/rebalancer.cc@257
PS1, Line 257:
in


http://gerrit.cloudera.org:8080/#/c/11748/1/src/kudu/tools/rebalancer.cc@289
PS1, Line 289: // TODO(aserbin): it would be nice to run these rebalancers in 
parallel
Yeah, I think that's a good idea. But also worth saving for a followup-once the 
core functionality it committed.


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

http://gerrit.cloudera.org:8080/#/c/11748/1/src/kudu/tools/rebalancer_tool-test.cc@1170
PS1, Line 1170: This tests
These tests are


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

http://gerrit.cloudera.org:8080/#/c/11748/1/src/kudu/tools/tool_action_cluster.cc@107
PS1, Line 107: envolves
involves



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Will Berkeley 
Gerrit-Comment-Date: Mon, 22 Oct 2018 22:11:31 +
Gerrit-HasComments: Yes


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11748 )

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..


Patch Set 1: Verified+1

Unrelated flake in TraceEventCallbackTest.TraceEventCallbackAndRecordingDuration


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 22 Oct 2018 16:01:18 +
Gerrit-HasComments: No


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change.  ( 
http://gerrit.cloudera.org:8080/11748 )

Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..


Removed reviewer Kudu Jenkins with the following votes:

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: deleteReviewer
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin 
Gerrit-Reviewer: Alexey Serbin 


[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-20 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/11748


Change subject: [rebalancer] location-aware rebalancer (part 7/n)
..

[rebalancer] location-aware rebalancer (part 7/n)

Added PolicyFixer and integrated the cross-location location
balancing algorithm.

Added one integration test as well, but it's disabled for a now since
it requires a tablet server's location to be reported in
KsckServerHealthSummary.  If that piece is in place, the test passes.

More integration tests will be added in a follow-up commit.

Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
---
M src/kudu/tools/rebalancer.cc
M src/kudu/tools/rebalancer.h
M src/kudu/tools/rebalancer_tool-test.cc
M src/kudu/tools/tool_action_cluster.cc
4 files changed, 709 insertions(+), 50 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9ace790aad1c1a4605ef90f6df2104f4a228a5b5
Gerrit-Change-Number: 11748
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin