Skip to content

br: update limitation#4265

Merged
ti-srebot merged 3 commits into
pingcap:masterfrom
3pointer:br_limitation
Aug 14, 2020
Merged

br: update limitation#4265
ti-srebot merged 3 commits into
pingcap:masterfrom
3pointer:br_limitation

Conversation

@3pointer
Copy link
Copy Markdown
Contributor

@3pointer 3pointer commented Aug 14, 2020

What is changed, added or deleted? (Required)

Update out-date limitation of br

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:
  • Other reference link(s):

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Have version specific changes
  • Might cause conflicts

@coocood
Copy link
Copy Markdown
Member

coocood commented Aug 14, 2020

LGTM

@ti-srebot
Copy link
Copy Markdown
Contributor

ti-srebot commented Aug 14, 2020

@coocood,Thanks for your review. However, LGTM is restricted to Reviewers or higher roles.See the corresponding SIG page for more information. Related SIGs: docs(slack).

TomShawn
TomShawn reviewed Aug 14, 2020
View reviewed changes
Comment thread br/backup-and-restore-tool.md Outdated
TomShawn
TomShawn approved these changes Aug 14, 2020
View reviewed changes
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

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Aug 14, 2020
@TomShawn TomShawn added needs-cherry-pick-4.0 require-LGT1 Indicates that the PR requires an LGTM. size/small Changes of a small size. status/can-merge Indicates a PR has been approved by a committer. translation/doing This PR’s assignee is translating this PR. labels Aug 14, 2020
@ti-srebot
Copy link
Copy Markdown
Contributor

ti-srebot commented Aug 14, 2020

/run-all-tests

@ti-srebot ti-srebot merged commit 9bf71ba into pingcap:master Aug 14, 2020
ti-srebot pushed a commit to ti-srebot/docs-cn that referenced this pull request Aug 14, 2020
@3pointer @ti-srebot
cherry pick pingcap#4265 to release-4.0
fbb8800 
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot mentioned this pull request Aug 14, 2020
Merged
9 tasks
@ti-srebot
Copy link
Copy Markdown
Contributor

ti-srebot commented Aug 14, 2020

cherry pick to release-4.0 in PR #4271

TomShawn pushed a commit that referenced this pull request Aug 14, 2020
@ti-srebot @3pointer
cherry pick #4265 to release-4.0 (#4271)
93f95ad 
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>

Co-authored-by: 3pointer <luancheng@pingcap.com>
@TomShawn TomShawn self-assigned this Sep 4, 2020
@TomShawn TomShawn mentioned this pull request Sep 4, 2020
Merged
9 tasks
@ran-huang ran-huang added translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. and removed translation/doing This PR’s assignee is translating this PR. labels Sep 4, 2020
This was referenced Sep 4, 2020
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@TomShawn TomShawn TomShawn approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@TomShawn TomShawn

Labels

require-LGT1 Indicates that the PR requires an LGTM. size/small Changes of a small size. status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@3pointer @coocood @ti-srebot @TomShawn @ran-huang