-
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
*: update keyword auto_increment to CAPS #2181
*: update keyword auto_increment to CAPS #2181
Conversation
Please help review the changes in DM-related files, especially for |
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.
rest LGTM
dev/reference/tools/data-migration/usage-scenarios/shard-merge.md
Outdated
Show resolved
Hide resolved
dev/reference/tools/data-migration/usage-scenarios/shard-merge.md
Outdated
Show resolved
Hide resolved
v3.0/reference/tools/data-migration/usage-scenarios/shard-merge.md
Outdated
Show resolved
Hide resolved
v3.1/reference/tools/data-migration/usage-scenarios/shard-merge.md
Outdated
Show resolved
Hide resolved
v3.1/reference/tools/data-migration/usage-scenarios/shard-merge.md
Outdated
Show resolved
Hide resolved
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
Co-Authored-By: Keke Yi <[email protected]>
Co-Authored-By: Keke Yi <[email protected]>
Co-Authored-By: Keke Yi <[email protected]>
Co-Authored-By: Keke Yi <[email protected]>
Co-Authored-By: Keke Yi <[email protected]>
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
/run-all-tests |
What is changed, added or deleted?
AUTO_INCREMENT: update keyword auto_increment to CAPS
What is the related PR or file link(s)?
pingcap/docs#1750
Which version does your change affect?
ALL