Skip to content

br: remove backup and restore principle (#3706)#3802

Merged
ti-srebot merged 2 commits into
pingcap:release-3.1from
ti-srebot:release-3.1-8cce37eca800
Jun 28, 2020
Merged

br: remove backup and restore principle (#3706)#3802
ti-srebot merged 2 commits into
pingcap:release-3.1from
ti-srebot:release-3.1-8cce37eca800

Conversation

@ti-srebot
Copy link
Copy Markdown
Contributor

cherry-pick #3706 to release-3.1


What is changed, added or deleted? (Required)

Remove unnecessary BR principles.

Which TiDB version(s) do your changes apply to? (Required)

  • master (the latest development version)
  • v4.0 (TiDB 4.0 versions)
  • v3.1 (TiDB 3.1 versions)
  • v3.0 (TiDB 3.0 versions)
  • v2.1 (TiDB 2.1 versions)

What is the related PR or file link(s)?

  • This PR is translated from: N/A
  • Other reference link(s): N/A

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot added the type/cherry-pick-for-release-3.1 This PR is cherry-picked to release-3.1 from a source PR. label Jun 28, 2020
@ti-srebot ti-srebot requested review from 3pointer and TomShawn June 28, 2020 09:20
@ti-srebot
Copy link
Copy Markdown
Contributor Author

@overvenus please accept the invitation then you can push to the cherry-pick pull requests.
https://github.com/ti-srebot/docs-cn/invitations

Comment thread br/backup-and-restore-tool.md Outdated
> 聚集这些备份数据可能会造成数据冗余和运维上的麻烦,而且在不聚集这些数据便直接恢复的时候会遇到颇为迷惑的 `SST file not found` 报错。
>
> 建议在各个节点挂载 NFS 网盘,或者直接备份到 `S3` 对象存储中。
>>>>>>> 8cce37e... br: remove backup and restore principle (#3706)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@overvenus Please resolve the conflict.

Comment thread br/backup-and-restore-tool.md Outdated
Comment thread br/backup-and-restore-tool.md Outdated
Copy link
Copy Markdown
Contributor

@TomShawn TomShawn left a comment

Choose a reason for hiding this comment

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

LGTM

@TomShawn
Copy link
Copy Markdown
Contributor

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Jun 28, 2020
@ti-srebot
Copy link
Copy Markdown
Contributor Author

/run-all-tests

@ti-srebot ti-srebot merged commit 100c1a5 into pingcap:release-3.1 Jun 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/can-merge Indicates a PR has been approved by a committer. type/cherry-pick-for-release-3.1 This PR is cherry-picked to release-3.1 from a source PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants