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

storage: extend the ExternalStorage interface and implements (#446)#462

Merged
3pointer merged 1 commit intopingcap:release-4.0from
glorv:cp-446
Aug 17, 2020
Merged

storage: extend the ExternalStorage interface and implements (#446)#462
3pointer merged 1 commit intopingcap:release-4.0from
glorv:cp-446

Conversation

@glorv
Copy link
Collaborator

@glorv glorv commented Aug 17, 2020

What problem does this PR solve?

cherry-pick #446 to release-4.0

What is changed and how it works?

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Code changes

  • Has exported function/method change
  • Has exported variable/fields change
  • Has interface methods change
  • Has persistent data change

Side effects

  • Possible performance regression
  • Increased code complexity
  • Breaking backward compatibility

Related changes

  • Need to cherry-pick to the release branch
  • Need to update the documentation

Release Note

…ap#446)

* update storage

* update

* fix

* export s3 flags

* fix local path

* fix s3 reader

* fix range offset

* fix the req range errors

* fix comments

* fix review comments

* fix some comments

* update

* fix

* update

Co-authored-by: ti-srebot <66930949+ti-srebot@users.noreply.github.com>
@glorv glorv requested review from 3pointer and kennytm August 17, 2020 11:39
Copy link
Collaborator

@3pointer 3pointer left a comment

Choose a reason for hiding this comment

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

LGTM

@ti-srebot ti-srebot added the status/LGT1 LGTM1 label Aug 17, 2020
@kennytm
Copy link
Collaborator

kennytm commented Aug 17, 2020

can we pick #456 here as well? or do it in a separate PR?

@3pointer
Copy link
Collaborator

can we pick #456 here as well? or do it in a separate PR?

I'll do it in another PR after this PR merged

@ti-srebot ti-srebot removed the status/LGT1 LGTM1 label Aug 17, 2020
@ti-srebot ti-srebot added the status/LGT2 LGTM2 label Aug 17, 2020
@3pointer 3pointer merged commit 5a0ee9a into pingcap:release-4.0 Aug 17, 2020
@glorv glorv deleted the cp-446 branch April 8, 2021 04:01
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