Closed lichunzhu closed 3 years ago
/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 /cc @reviewer
in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer
in the comment to help you merge this pull request.
The full list of commands accepted by this bot can be found here.
/merge
This pull request has been accepted and is ready to merge.
/hold
/unhold
/merge
This pull request has been accepted and is ready to merge.
In response to a cherrypick label: new pull request created: #287.
In response to a cherrypick label: new pull request created: #288.
What problem does this PR solve?
https://github.com/pingcap/br/issues/898
storage.Create
has been deprecated... so let's replace the remaining two calls from Dumpling.What is changed and how it works?
storage.Create
tostorage.New
with sendCreds=false and skipCheckPath=true.Check List
Tests
Side effects
Related changes
Release note