-
Notifications
You must be signed in to change notification settings - Fork 102
backup: allow backup tolerate minor TiKV failure #997
Conversation
/lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rest LGTM
@@ -51,6 +53,11 @@ func (push *pushDown) pushBackup( | |||
|
|||
// Push down backup tasks to all tikv instances. | |||
res := rtree.NewRangeTree() | |||
failpoint.Inject("noop-backup", func(_ failpoint.Value) { | |||
log.Warn("skipping normal backup, jump to fine-grained backup, meow :3", logutil.Key("start-key", req.StartKey), logutil.Key("end-key", req.EndKey)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😼
Co-authored-by: kennytm <kennytm@gmail.com>
/lgtm |
[REVIEW NOTIFICATION] This pull request has been approved by:
To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by writing |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: 0106164
|
/run-integration-tests |
1 similar comment
/run-integration-tests |
/run-integration-tests |
1 similar comment
/run-integration-tests |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-5.0 in PR #1019 |
What problem does this PR solve?
fixed #980
What is changed and how it works?
ignore all ‘failed to connect to store’ errors during backup.
Check List
Tests
Release Note