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

gcs: compatible with old gcs bug (#677) #688

Merged
merged 4 commits into from
Jan 11, 2021

Conversation

ti-srebot
Copy link
Contributor

@ti-srebot ti-srebot commented Jan 11, 2021

cherry-pick #677 to release-4.0
You can switch your code base to this Pull Request by using git-extras:

# In br repo:
git pr 688

After apply modifications, you can push your change to this PR via:

git push git@github.com:ti-srebot/br.git pr/688:ti-srebot:release-4.0-3de76b2ec3f2

What problem does this PR solve?

solve #675

What is changed and how it works?

for case 1: add a check before Read & Write meta file, decide which prefix to use.
for case 2: check whether sst file is in Prefix(after #647) or in PrefixSlash(before #647), adjust proper prefix for TiKV request.

Check List

Tests

  • Integration test

Related changes

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

Release Note

  • Fix the issue that br 4.0.9 cannot restore br 4.0.8 gcs backup files.

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@3pointer
Copy link
Collaborator

/run-all-tests

1 similar comment
@3pointer
Copy link
Collaborator

/run-all-tests

@3pointer
Copy link
Collaborator

[2021-01-11T11:30:05.513Z] [2021/01/11 19:29:49.775 +08:00] [FATAL] [server.rs:846] ["0.0.0.0_20170 already in use, maybe another instance is binding with this address."]

@3pointer
Copy link
Collaborator

/run-all-tests

@3pointer
Copy link
Collaborator

/run-all-tests

@overvenus
Copy link
Member

LGTM

@ti-srebot ti-srebot added the status/LGT1 LGTM1 label Jan 11, 2021
@kennytm
Copy link
Collaborator

kennytm commented Jan 11, 2021

Lgtm

@ti-srebot ti-srebot added status/LGT2 LGTM2 and removed status/LGT1 LGTM1 labels Jan 11, 2021
@overvenus
Copy link
Member

/merge

@kennytm kennytm merged commit 5044b6f into pingcap:release-4.0 Jan 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants