schedule: operator limit for stores #1474
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## master #1474 +/- ##
==========================================
- Coverage 68.07% 67.51% -0.56%
==========================================
Files 159 159
Lines 15843 16134 +291
==========================================
+ Hits 10785 10893 +108
- Misses 4100 4269 +169
- Partials 958 972 +14
Continue to review full report at Codecov.
|
rleungx
force-pushed
the
limit-store
branch
2 times, most recently
from
a7df917 to
a1a1921
Compare
rleungx
added
the
do-not-merge/work-in-progress
rleungx
added
DNM
and removed
do-not-merge/work-in-progress
rleungx
force-pushed
the
limit-store
branch
2 times, most recently
from
2a41999 to
2df2083
Compare
disksing
reviewed
Apr 8, 2019
rleungx
force-pushed
the
limit-store
branch
2 times, most recently
from
e93bf61 to
aa79ff9
Compare
rleungx
force-pushed
the
limit-store
branch
9 times, most recently
from
7643be0 to
5713c77
Compare
nolouch
reviewed
May 8, 2019
rleungx
requested review from
nolouch and
disksing
and removed request for
nolouch
disksing
reviewed
May 21, 2019
disksing
reviewed
May 21, 2019
|
/rebuild |
shafreeck
approved these changes
May 22, 2019
Signed-off-by: nolouch <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>
|
/rebuild |
nolouch
added a commit
that referenced
this pull request
May 23, 2019
* tiny clean up code (#1526) Signed-off-by: Ryan Leung <[email protected]> * config: enable use region storage by default (#1524) * config: default enable use region storage * schedulers: let hot region balance not affect by balance-region-scheduler-limit (#1522) Signed-off-by: nolouch <[email protected]> * log: do not add stack to error log (#1532) * *: replace gofail with pingcap/failpoint (#1534) * *: replace gofail with pingcap/failpoint Signed-off-by: disksing <[email protected]> * fix typo Signed-off-by: disksing <[email protected]> * fix typo Signed-off-by: disksing <[email protected]> * Update server/cluster_test.go Co-Authored-By: Lonng <[email protected]> * Update server/tso.go Co-Authored-By: Lonng <[email protected]> * Update server/tso.go Co-Authored-By: Lonng <[email protected]> * check enable failpoint result Signed-off-by: disksing <[email protected]> * Update server/cluster.go Co-Authored-By: Lonng <[email protected]> * server: set timeout for MoveLeader (#1533) * server: set timeout for MoveLeader Signed-off-by: disksing <[email protected]> * client, server: add ScanRegions gRPC protocol support (#1535) * client, server: support ScanRegions gRPC protocol Signed-off-by: disksing <[email protected]> * schedule: actively push operator (#1536) * schedule: actively push operator Signed-off-by: nolouch <[email protected]> * *: update some dead links (#1543) * update links Signed-off-by: Ryan Leung <[email protected]> * Add windows build script (#1540) Signed-off-by: Ana Hobden <[email protected]> * fix operator timeout metrics (#1541) Signed-off-by: Ryan Leung <[email protected]> * *:Rollback config in store when kv.persist failed (#1476) * tests: independent region sync test (#1545) * tests: independent syncer region test Signed-off-by: nolouch <[email protected]> * schedule: operator limit for stores (#1474) * add store limit for scheduling Signed-off-by: Ryan Leung <[email protected]> * statistic: add the statistic of flow (#1548) * store_statistic: add the statistic of flow Signed-off-by: nolouch <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What problem does this PR solve?
We can not control the operators' cost in a store.
What is changed and how it works?
This PR adds an overloaded filter for stores. If the operators' cost in a store exceeds
StoreMaxScheduleCost, the store will be filtered.Also if the total cost of current running operators exceeds
MaxScheduleCost, the newly created operator will be not added into running operators.Check List
Tests