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

v2.1, v3.0, dev: fix inconsistent word usage #1287

Merged
merged 1 commit into from
Jun 28, 2019

Conversation

yikeke
Copy link
Contributor

@yikeke yikeke commented Jun 27, 2019

I found that we used two different names for the SQL statement - CREATE TABLE LIKE and CREATE TABLE AS in our documents. According to the usage examples in the document, I think we should use the former one.
@morgo PTAL

@morgo morgo self-requested a review June 27, 2019 13:13
Copy link
Contributor

@morgo morgo left a comment

Choose a reason for hiding this comment

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

LGTM

For context: CREATE TABLE AS is valid MySQL syntax. We have a PR to add it, but it's currently stalled: pingcap/tidb#7787 It is on the mysql-compatibility page.

@yikeke
Copy link
Contributor Author

yikeke commented Jun 28, 2019

LGTM

For context: CREATE TABLE AS is valid MySQL syntax. We have a PR to add it, but it's currently stalled: pingcap/tidb#7787 It is on the mysql-compatibility page.

Okay, thanks for letting me know.

@yikeke yikeke merged commit 1923320 into pingcap:master Jun 28, 2019
@yikeke yikeke deleted the update-word branch June 28, 2019 03:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants