-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
*: add smooth upgrade TiDB introduction doc #14013
Conversation
[REVIEW NOTIFICATION] This pull request has been approved by:
To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
16f11de
to
d25e45a
Compare
5db914c
to
a086164
Compare
smooth-upgrade-tidb.md
Outdated
|
||
* BR,Lightning (Physical Import Mode) 和通过 ingest 方式导入数据等组件:升级过程中不能处理这类组件的操作,是因为这些操作中可能将 paused 状态的 DDL 拷贝到 TiDB。但是此状态的 DDL 不能自动 resume,可能导致后续 DDL 卡住的情况。 | ||
|
||
* DM,Lightning (Logical Import Mode) 和 TiCDC 等组件。即在升级过程中,通过此类组件导入 SQL 到 TiDB,且其中有 DDL 操作,那么会导致此导入操作阻塞,此外可能存在未定义错误。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* DM,Lightning (Logical Import Mode) 和 TiCDC 等组件。即在升级过程中,通过此类组件导入 SQL 到 TiDB,且其中有 DDL 操作,那么会导致此导入操作阻塞,此外可能存在未定义错误。 | |
* DM 和 TiCDC 等组件。即在升级过程中,通过此类组件导入 SQL 到 TiDB,且其中有 DDL 操作,那么会导致此导入操作阻塞,此外可能存在未定义错误。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
尽量不要出现 lightning 的字样,另外这种导入后续不是 logical 导入。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
什么原因?“Lightning (Physical Import Mode)” 要改吗?
PTAL @Benjamin2037 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PTAL @ran-huang |
PTAL @qiancai @ran-huang |
1 similar comment
PTAL @qiancai @ran-huang |
Co-authored-by: Grace Cai <qqzczy@126.com>
Co-authored-by: Grace Cai <qqzczy@126.com>
Co-authored-by: Grace Cai <qqzczy@126.com>
Co-authored-by: Ran <huangran.alex@gmail.com>
Co-authored-by: Grace Cai <qqzczy@126.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/verify |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: 50278eb
|
First-time contributors' checklist
What is changed, added or deleted? (Required)
This PR is used to add introduction doc for smooth upgrade.
Which TiDB version(s) do your changes apply to? (Required)
Tips for choosing the affected version(s):
By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.
For details, see tips for choosing the affected versions (in Chinese).
What is the related PR or file link(s)?
Do your changes match any of the following descriptions?