Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

restore: add scatter region time limit#129

Merged
overvenus merged 1 commit intopingcap:masterfrom
5kbpers:fix-scatter
Dec 26, 2019
Merged

restore: add scatter region time limit#129
overvenus merged 1 commit intopingcap:masterfrom
5kbpers:fix-scatter

Conversation

@5kbpers
Copy link
Contributor

@5kbpers 5kbpers commented Dec 26, 2019

Signed-off-by: 5kbpers tangminghua@pingcap.com

Relate to pingcap/tidb-tools#305

Signed-off-by: 5kbpers <tangminghua@pingcap.com>
@codecov
Copy link

codecov bot commented Dec 26, 2019

Codecov Report

Merging #129 into master will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #129      +/-   ##
==========================================
+ Coverage   72.53%   72.56%   +0.03%     
==========================================
  Files          33       33              
  Lines        3189     3189              
==========================================
+ Hits         2313     2314       +1     
+ Misses        588      587       -1     
  Partials      288      288
Impacted Files Coverage Δ
pkg/backup/client.go 71.08% <0%> (+0.21%) ⬆️

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 d934b16...5f0c6e8. Read the comment docs.

Copy link
Collaborator

@kennytm kennytm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kennytm kennytm added the status/LGT1 LGTM1 label Dec 26, 2019
Copy link
Member

@overvenus overvenus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@overvenus overvenus merged commit b362e1d into pingcap:master Dec 26, 2019
@zhangjinpeng87
Copy link

Bad PR description.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants