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

[new-hotspot-scheduler] merge master #1752

Merged
merged 26 commits into from
Sep 11, 2019

Conversation

Luffbee
Copy link
Contributor

@Luffbee Luffbee commented Sep 11, 2019

merge master into new-hotspot-scheduler.

rleungx and others added 26 commits August 19, 2019 15:41
Signed-off-by: Ryan Leung <rleungx@gmail.com>
* test: fix region syncer test

Signed-off-by: disksing <i@disksing.com>
Signed-off-by: cwen0 <cwenyin0@gmail.com>
Signed-off-by: disksing <i@disksing.com>
Signed-off-by: Ryan Leung <rleungx@gmail.com>
…#1659)

* schedule: Do not send an operator of a region wth a stale epoch

Signed-off-by: Shafreeck Sea <shafreeck@gmail.com>

* schedule: check the version changed by the operator self

Signed-off-by: Shafreeck Sea <shafreeck@gmail.com>

* schedule: fix unit test

Signed-off-by: Shafreeck Sea <shafreeck@gmail.com>

* schedule: fix to avoid dispatching a stale opstep

Signed-off-by: Shafreeck Sea <shafreeck@gmail.com>

* dispatch: refactor "ConsumeConfVer() int" to "ExpectConfVerChange() bool"

Signed-off-by: Shafreeck Sea <shafreeck@gmail.com>

* dispatch: fix typo in comment

Signed-off-by: Shafreeck Sea <shafreeck@gmail.com>

* fix typo

Co-Authored-By: Ryan Leung <rleungx@gmail.com>

* dispatch: fix unittest

Signed-off-by: Shafreeck Sea <shafreeck@gmail.com>

* dispatch: refine format

Signed-off-by: Shafreeck Sea <shafreeck@gmail.com>
Signed-off-by: Ryan Leung <rleungx@gmail.com>
Signed-off-by: Ryan Leung <rleungx@gmail.com>
Signed-off-by: Ryan Leung <rleungx@gmail.com>
Signed-off-by: Ryan Leung <rleungx@gmail.com>
Signed-off-by: nolouch <nolouch@gmail.com>
…th smaller size (tikv#1723)

Signed-off-by: disksing <i@disksing.com>
Signed-off-by: Ryan Leung <rleungx@gmail.com>
Signed-off-by: Ryan Leung <rleungx@gmail.com>
Signed-off-by: jiyingtk <jiyingtk@mail.ustc.edu.cn>
Signed-off-by: disksing <i@disksing.com>
@codecov-io
Copy link

Codecov Report

Merging #1752 into new-hotspot-scheduler will increase coverage by 0.11%.
The diff coverage is 69.56%.

Impacted file tree graph

@@                    Coverage Diff                    @@
##           new-hotspot-scheduler    #1752      +/-   ##
=========================================================
+ Coverage                  75.55%   75.67%   +0.11%     
=========================================================
  Files                        161      161              
  Lines                      16262    16278      +16     
=========================================================
+ Hits                       12287    12318      +31     
+ Misses                      2935     2926       -9     
+ Partials                    1040     1034       -6
Impacted Files Coverage Δ
server/api/region.go 68.83% <100%> (+0.29%) ⬆️
server/core/region_option.go 84.25% <100%> (+0.92%) ⬆️
server/schedule/operator_controller.go 90.42% <100%> (ø) ⬆️
server/statistics/hot_cache.go 59.85% <36.36%> (-1.46%) ⬇️
server/schedulers/random_merge.go 61.53% <0%> (-5.13%) ⬇️
server/grpc_service.go 58.78% <0%> (-0.22%) ⬇️
server/cluster.go 84.42% <0%> (+0.25%) ⬆️
server/schedule/operator/operator.go 84.72% <0%> (+0.53%) ⬆️
client/client.go 69.76% <0%> (+0.86%) ⬆️
server/member/leader.go 77.04% <0%> (+1.53%) ⬆️
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ecc0661...d6618de. Read the comment docs.

@Luffbee Luffbee merged commit bcfcada into tikv:new-hotspot-scheduler Sep 11, 2019
@Luffbee Luffbee deleted the merge-master branch September 11, 2019 09:18
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.

9 participants