You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Yuqi Du (Code Review)" <ge...@cloudera.org> on 2022/07/28 03:21:30 UTC

[kudu-CR] [master] support turning on/off auto rebalancer at runtime

Hello Tidy Bot, Alexey Serbin, Yingchun Lai, Kudu Jenkins, 

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

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

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

Change subject: [master] support turning on/off auto rebalancer at runtime
......................................................................

[master] support turning on/off auto rebalancer at runtime

kudu-master supports auto-rebalancer, but it must be decided whether
enable it before kudu master starting.

This patch supports to turn on/off `auto rebalancer` when
kudu-master running by changing gflag --auto_rebalancing_enabled.

There are many independent, idle and isolated threads(which do period tasks)
they don't participant in threadpool. We can use less threads, unified threadpool
to manager them. Unified code style after more deeper refactoring.
The patch is the first refactoring attempt.

Change-Id: Ifbcea6392b783843f89e019f1141dd8905366de5
---
M src/kudu/master/auto_rebalancer-test.cc
M src/kudu/master/auto_rebalancer.cc
M src/kudu/master/auto_rebalancer.h
M src/kudu/master/catalog_manager.cc
M src/kudu/master/catalog_manager.h
5 files changed, 155 insertions(+), 77 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/79/18679/25
-- 
To view, visit http://gerrit.cloudera.org:8080/18679
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ifbcea6392b783843f89e019f1141dd8905366de5
Gerrit-Change-Number: 18679
Gerrit-PatchSet: 25
Gerrit-Owner: Yuqi Du <sh...@gmail.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yingchun Lai <ac...@gmail.com>
Gerrit-Reviewer: Yuqi Du <sh...@gmail.com>