-
Notifications
You must be signed in to change notification settings - Fork 102
backup: set concurrency to 1 when ratelimit enabled (#1015) #1025
backup: set concurrency to 1 when ratelimit enabled (#1015) #1025
Conversation
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
/run-all-tests |
/lgtm |
/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: 1358222
|
/run-integration-test CI network issues. https://internal.pingcap.net/idc-jenkins/blue/organizations/jenkins/br_ghpr_unit_and_integration_test/detail/br_ghpr_unit_and_integration_test/11518/pipeline |
@ti-srebot: Your PR was out of date, I have automatically updated it for you. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository. |
cherry-pick #1015 to release-4.0
You can switch your code base to this Pull Request by using git-extras:
# In br repo: git pr https://github.com/pingcap/br/pull/1025
After apply modifications, you can push your change to this PR via:
What problem does this PR solve?
Fix? #1007
What is changed and how it works?
When
--ratelimit
was set, degenerate to sequential backing up.Check List
Tests
(See backup:
--ratelimit
doesn't work as intended #1007)Release Note
--ratelimit
.