Skip to content

[cherry-pick] br: add use constraint with CDC and drainer (#3750)#4184

Merged
TomShawn merged 5 commits into
pingcap:release-3.1from
YuJuncen:cherry-pick-3750
Aug 4, 2020
Merged

[cherry-pick] br: add use constraint with CDC and drainer (#3750)#4184
TomShawn merged 5 commits into
pingcap:release-3.1from
YuJuncen:cherry-pick-3750

Conversation

@YuJuncen
Copy link
Copy Markdown
Contributor

@YuJuncen YuJuncen commented Aug 4, 2020

Cherry-picking #3750

===

What is changed, added or deleted? (Required)

现今 CDC 和 Drainer 无法同步 BR 所恢复的数据,同时 BR 恢复到上游集群可能会让 CDC / Drainer 工作异常。

于是在文档中标注出来。

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)

* br: add use constraint with CDC and drainer

* br: fix a bad format

* Apply suggestions from code review

Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>

* Update backup-and-restore-faq.md

Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
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 4, 2020
@TomShawn TomShawn added type/cherry-pick-for-release-3.1 This PR is cherry-picked to release-3.1 from a source PR. status/can-merge Indicates a PR has been approved by a committer. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Aug 4, 2020
@TomShawn TomShawn merged commit b5fffbc into pingcap:release-3.1 Aug 4, 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