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

schedule: fix panic issue after delete the rule (#2513) #2515

Merged
merged 3 commits into from
Jun 5, 2020

Conversation

sre-bot
Copy link
Contributor

@sre-bot sre-bot commented Jun 5, 2020

cherry-pick #2513 to release-4.0


What problem does this PR solve?

Fix #2478

What is changed and how it works?

check the length.

Check List

Tests

  • Manual test (add detailed scripts or steps below)

Release note

  • fix the panic issue after deleting the rule

Signed-off-by: nolouch <nolouch@gmail.com>
Signed-off-by: nolouch <nolouch@gmail.com>
@codecov-commenter
Copy link

Codecov Report

Merging #2515 into release-4.0 will increase coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##           release-4.0    #2515   +/-   ##
============================================
  Coverage        77.10%   77.10%           
============================================
  Files              204      204           
  Lines            21885    21887    +2     
============================================
+ Hits             16874    16877    +3     
+ Misses            3730     3727    -3     
- Partials          1281     1283    +2     
Impacted Files Coverage Δ
server/schedule/placement/fit.go 89.14% <0.00%> (-1.41%) ⬇️
server/cluster/coordinator.go 74.52% <0.00%> (-2.20%) ⬇️
server/server.go 77.53% <0.00%> (-0.79%) ⬇️
server/grpc_service.go 58.99% <0.00%> (-0.39%) ⬇️
server/schedulers/adjacent_region.go 76.59% <0.00%> (+1.06%) ⬆️
pkg/etcdutil/etcdutil.go 84.05% <0.00%> (+1.44%) ⬆️
server/tso/tso.go 83.21% <0.00%> (+2.18%) ⬆️
server/member/leader.go 74.31% <0.00%> (+2.33%) ⬆️
server/region_syncer/client.go 84.73% <0.00%> (+3.05%) ⬆️
server/schedulers/base_scheduler.go 69.56% <0.00%> (+8.69%) ⬆️

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 bbf1354...04b09b9. Read the comment docs.

@nolouch
Copy link
Contributor

nolouch commented Jun 5, 2020

/approve

@nolouch
Copy link
Contributor

nolouch commented Jun 5, 2020

/merge

@lhy1024
Copy link
Contributor

lhy1024 commented Jun 5, 2020

/merge

@sre-bot sre-bot added the status/can-merge Indicates a PR has been approved by a committer. label Jun 5, 2020
@sre-bot
Copy link
Contributor Author

sre-bot commented Jun 5, 2020

/run-all-tests

@sre-bot
Copy link
Contributor Author

sre-bot commented Jun 5, 2020

@sre-bot merge failed.

@lhy1024
Copy link
Contributor

lhy1024 commented Jun 5, 2020

/merge

@sre-bot
Copy link
Contributor Author

sre-bot commented Jun 5, 2020

/run-all-tests

@sre-bot
Copy link
Contributor Author

sre-bot commented Jun 5, 2020

@sre-bot merge failed.

@nolouch
Copy link
Contributor

nolouch commented Jun 5, 2020

/merge

@sre-bot
Copy link
Contributor Author

sre-bot commented Jun 5, 2020

/run-all-tests

@sre-bot sre-bot merged commit 30f0b01 into tikv:release-4.0 Jun 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/can-merge Indicates a PR has been approved by a committer. type/bugfix This PR fixes a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants