Skip to content

sql: add explanation for different versions#2076

Merged
sre-bot merged 3 commits into
pingcap:masterfrom
toutdesuite:add-explaination
Mar 25, 2020
Merged

sql: add explanation for different versions#2076
sre-bot merged 3 commits into
pingcap:masterfrom
toutdesuite:add-explaination

Conversation

@toutdesuite
Copy link
Copy Markdown
Contributor

@toutdesuite toutdesuite commented Mar 23, 2020

What is changed, added or deleted? (Required)

Explain a problem which may take attention on upgrade

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

  • master (the latest development version, including v4.0 changes for now)
  • v3.1 (TiDB 3.1 versions)
  • v3.0 (TiDB 3.0 versions)
  • v2.1 (TiDB 2.1 versions)

If you select two or more versions from above, to trigger the bot to cherry-pick this PR to your desired release version branch(es), you must add corresponding labels such as needs-cherry-pick-3.1, needs-cherry-pick-3.0, and needs-cherry-pick-2.1.

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

@toutdesuite toutdesuite requested review from imtbkcat and yikeke March 23, 2020 12:07
@toutdesuite toutdesuite added needs-cherry-pick-3.0 translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn. labels Mar 23, 2020
Comment thread reference/sql/statements/use.md Outdated
## MySQL compatibility

This statement is understood to be fully compatible with MySQL. Any compatibility differences should be [reported via an issue](/report-issue.md) on GitHub.
For TiDB 2.0, you can use the `USE` statement to access any database. For TiDB 3.0 or later versions, it checks whether you have access to the database when using the `USE` statement. This statement is understood to be fully compatible with MySQL. Any compatibility differences should be [reported via an issue](/report-issue.md) on GitHub.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
For TiDB 2.0, you can use the `USE` statement to access any database. For TiDB 3.0 or later versions, it checks whether you have access to the database when using the `USE` statement. This statement is understood to be fully compatible with MySQL. Any compatibility differences should be [reported via an issue](/report-issue.md) on GitHub.
In TiDB 2.0 versions, you can use `USE` statements to access any database. In TiDB 3.0 or later versions, TiDB checks whether you have the privilege to access the database when you use `USE` statements.
Currently, `USE` statements are understood to be fully compatible with MySQL. Any compatibility differences should be [reported via an issue](/report-issue.md) on GitHub.

Copy link
Copy Markdown
Contributor

@yikeke yikeke left a comment

Choose a reason for hiding this comment

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

LGTM

@yikeke
Copy link
Copy Markdown
Contributor

yikeke commented Mar 24, 2020

PTAL @imtbkcat

Copy link
Copy Markdown

@imtbkcat imtbkcat left a comment

Choose a reason for hiding this comment

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

LGTM

@imtbkcat imtbkcat added the status/LGT2 Indicates that a PR has LGTM 2. label Mar 25, 2020
@yikeke yikeke added the status/can-merge Indicates a PR has been approved by a committer. label Mar 25, 2020
@sre-bot
Copy link
Copy Markdown
Contributor

sre-bot commented Mar 25, 2020

Your auto merge job has been accepted, waiting for 1931

@sre-bot
Copy link
Copy Markdown
Contributor

sre-bot commented Mar 25, 2020

/run-all-tests

@sre-bot sre-bot merged commit 1eed18d into pingcap:master Mar 25, 2020
sre-bot pushed a commit to sre-bot/docs that referenced this pull request Mar 25, 2020
@sre-bot
Copy link
Copy Markdown
Contributor

sre-bot commented Mar 25, 2020

cherry pick to release-3.0 in PR #2092

@sre-bot
Copy link
Copy Markdown
Contributor

sre-bot commented Mar 25, 2020

cherry pick to release-3.1 in PR #2093

@toutdesuite toutdesuite deleted the add-explaination branch March 25, 2020 07:46
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. status/LGT2 Indicates that a PR has LGTM 2. translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants