Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

zh, en: add primary/unique key check description #223

Merged
merged 10 commits into from
Aug 17, 2020

Conversation

GMHDBJD
Copy link
Collaborator

@GMHDBJD GMHDBJD commented Aug 13, 2020

What is changed, added or deleted? (Required)

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

  • master (the latest development version, including v2.0 changes for now)
  • v1.0 (TiDB DM 1.0 versions)

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

dm#887 dm#870

  • 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

@GMHDBJD
Copy link
Collaborator Author

GMHDBJD commented Aug 13, 2020

Copy link
Member

@csuzhangxc csuzhangxc left a comment

Choose a reason for hiding this comment

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

rest LGTM

zh/precheck.md Outdated
@@ -43,6 +43,8 @@ aliases: ['/docs-cn/tidb-data-migration/dev/precheck/']
- TiDB 不支持外键
- 字符集的兼容性不同,详见 [TiDB 支持的字符集](https://pingcap.com/docs-cn/dev/reference/sql/character-set/)

DM 还会检查上游表中是否存在主键或 Unique 约束, 在 v1.0.7 版本引入
Copy link
Member

Choose a reason for hiding this comment

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

统一一下 主键Unique 的中英文表述?比如直接说 主键或唯一键

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

address in 27d4eb3

@ireneontheway ireneontheway added needs-cherry-pick-release-1.0 Should cherry pick this PR to release-1.0 branch. translation/no-need The changes in this PR don't need to be translated. labels Aug 13, 2020
@ireneontheway ireneontheway self-requested a review August 13, 2020 09:17
en/precheck.md Outdated Show resolved Hide resolved
GMHDBJD and others added 2 commits August 13, 2020 17:32
Copy link
Contributor

@ireneontheway ireneontheway 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
Copy link
Contributor

@ireneontheway,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).

zh/precheck.md Outdated Show resolved Hide resolved
Copy link
Member

@csuzhangxc csuzhangxc 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 13, 2020
Co-authored-by: ireneontheway <[email protected]>
@csuzhangxc
Copy link
Member

@TomShawn two LGTM now, PTAL

en/precheck.md Outdated Show resolved Hide resolved
zh/precheck.md Outdated Show resolved Hide resolved
@TomShawn TomShawn added require-LGT1 Indicates that the PR requires an LGTM. status/can-merge Indicates a PR has been approved by a committer. labels Aug 17, 2020
@ti-srebot
Copy link
Contributor

/run-all-tests

@ti-srebot ti-srebot merged commit 6093817 into pingcap:master Aug 17, 2020
ti-srebot pushed a commit to ti-srebot/docs-dm that referenced this pull request Aug 17, 2020
@ti-srebot
Copy link
Contributor

cherry pick to release-1.0 in PR #240

ti-srebot added a commit that referenced this pull request Aug 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-cherry-pick-release-1.0 Should cherry pick this PR to release-1.0 branch. require-LGT1 Indicates that the PR requires an LGTM. status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1. translation/no-need The changes in this PR don't need to be translated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants