Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

schedulers: add role config to shuffleRegionScheduler (#2219) #2235

Merged
merged 1 commit into from
Mar 12, 2020

Conversation

disksing
Copy link
Contributor

What problem does this PR solve?

cherry-pick #2219 to fix #2177

What is changed and how it works?

Add role configuration to shuffle-region-scheduler.

Use it to balance learners:

1. add shuffle-region-scheduler

pd-ctl scheduler add shuffle-region-scheduler

or

curl -XPOST -d '{"name": "shuffle-region-scheduler"}' http://pd:2379/pd/api/v1/schedulers

2. config roles

pd-ctl scheduler config shuffle-region-scheduler set-roles learner

or

curl -XPOST -d '["learner"]'  http://pd:2379/pd/api/v1/scheduler-config/shuffle-region-scheduler/roles

Check List

Tests

  • Unit test
  • Manual test

Code changes

Related changes

  • Need to cherry-pick to the release branch
  • Need to update the documentation

@disksing disksing requested review from rleungx and nolouch March 12, 2020 08:06
@disksing disksing added this to the v3.1.0-rc milestone Mar 12, 2020
@disksing
Copy link
Contributor Author

/run-all-test

@disksing
Copy link
Contributor Author

/run-all-tests

@disksing disksing merged commit aff95b9 into tikv:release-3.1 Mar 12, 2020
@disksing disksing deleted the cp2219 branch March 12, 2020 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants