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

scheduler: add grant hot region scheduler for test #2

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

bufferflies
Copy link
Owner

What problem does this PR solve?

add grant hot region scheduler for test

What is changed and how it works?

Check List

Tests

  • Unit test
  • Integration test

Code changes

Side effects

Related changes

Release note


Signed-off-by: bufferflies <[email protected]>
Signed-off-by: bufferflies <[email protected]>
Signed-off-by: bufferflies <[email protected]>
Signed-off-by: bufferflies <[email protected]>
Signed-off-by: bufferflies <[email protected]>
Signed-off-by: bufferflies <[email protected]>
Signed-off-by: bufferflies <[email protected]>
Signed-off-by: bufferflies <[email protected]>
Signed-off-by: bufferflies <[email protected]>
Signed-off-by: bufferflies <[email protected]>
Signed-off-by: bufferflies <[email protected]>
@codecov-commenter
Copy link

Codecov Report

Merging #2 (6a24eb1) into master (22d8802) will increase coverage by 0.44%.
The diff coverage is 76.84%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   74.38%   74.82%   +0.44%     
==========================================
  Files         260      265       +5     
  Lines       26468    27483    +1015     
==========================================
+ Hits        19688    20564     +876     
- Misses       4999     5093      +94     
- Partials     1781     1826      +45     
Flag Coverage Δ
unittests 74.82% <76.84%> (+0.44%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
server/config/config.go 75.05% <ø> (ø)
server/region_syncer/server.go 83.08% <ø> (-0.13%) ⬇️
server/schedule/opt/opts.go 100.00% <ø> (ø)
tests/cluster.go 80.78% <0.00%> (-0.88%) ⬇️
server/api/scheduler.go 48.63% <50.00%> (+0.04%) ⬆️
server/api/unsafe_operation.go 55.55% <55.55%> (ø)
server/schedule/operator_controller.go 84.72% <55.55%> (+0.71%) ⬆️
server/schedulers/grant_hot_region.go 56.28% <56.28%> (ø)
server/cluster/coordinator.go 75.10% <64.28%> (+1.78%) ⬆️
server/schedule/checker/priority_inspector.go 81.08% <66.66%> (ø)
... and 75 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 701b94f...6a24eb1. Read the comment docs.

@bufferflies bufferflies force-pushed the feature/shuffle-hot-region branch from 63585e4 to 6a24eb1 Compare November 9, 2021 06:17
Signed-off-by: bufferflies <[email protected]>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 9, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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.

2 participants