Skip to content

sql-mode: update compatibility#3192

Merged
3 commits merged into
masterfrom
unknown repository
Jul 10, 2020
Merged

sql-mode: update compatibility#3192
3 commits merged into
masterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jul 7, 2020

What is changed, added or deleted? (Required)

Fixes #1768

Also addresses a minor issue in mysql-compatibility where in my #3151 I said that SELECT INTO OUTFILE may not be implemented, but it since has been.

I also propose moving "Compatibility -> MySQL Compatibility" to "MySQL Compatibility" in the menu, to simplify navigation, and add myself as a reviewer to mysql compatibility areas.

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

  • master (the latest development version)
  • v4.0 (TiDB 4.0 versions)
  • v3.1 (TiDB 3.1 versions)
  • v3.0 (TiDB 3.0 versions)
  • v2.1 (TiDB 2.1 versions)

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

  • This PR is translated from:
  • Other reference link(s):

@yikeke
Copy link
Copy Markdown
Contributor

yikeke commented Jul 8, 2020

Note: mysql-compatibility.md is very different in versions below 4.0. You might need to resolve conflicts when this PR is cherry-picked to the lower versions.

@yikeke yikeke added status/PTAL This PR is ready for reviewing. translation/welcome Waits for a contributor to translate this PR and create a PR to the pingcap/docs-cn repository. labels Jul 8, 2020
@ghost
Copy link
Copy Markdown
Author

ghost commented Jul 8, 2020

Note: mysql-compatibility.md is very different in versions below 4.0. You might need to resolve conflicts when this PR is cherry-picked to the lower versions.

Yes, that's my plan :-)

@ti-srebot
Copy link
Copy Markdown
Contributor

@zz-jason, @scsldb, @kissmydb, @yikeke, PTAL.

Comment thread TOC.md
+ [Basic Features](/basic-features.md)
+ Compatibility
+ [MySQL Compatibility](/mysql-compatibility.md)
+ [MySQL Compatibility](/mysql-compatibility.md)
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.

In the future,maybe have other DB compatibility,so,i think just leave it as it is

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

If that happens in the future, we update the TOC and change it back :-)

Copy link
Copy Markdown
Member

@zz-jason zz-jason 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 Jul 10, 2020
@ti-srebot
Copy link
Copy Markdown
Contributor

@zz-jason,Thanks for your review.

Comment thread mysql-compatibility.md

TiDB is fully compatible with the MySQL 5.7 protocol and the common features and syntax of MySQL 5.7. The ecosystem tools for MySQL 5.7 (PHPMyAdmin, Navicat, MySQL Workbench, mysqldump, and Mydumper/myloader) and the MySQL client can be used for TiDB.

However, some features of MySQL are not supported. This could be because there is now a better way to solve the problem (such as XML functions superceded by JSON), security issues (such as `SELECT INTO OUTFILE`), or a lack of current demand versus effort required (such as stored procedures and functions). Some features might also be difficult to implement as a distributed system.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

select into outfile is supported in TiDB 4.0, we'd better not backport this PR to release 2.1, 3.0, and 3.1.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Understood. The 2.1-3.0 docs will still say this feature is not supported (it's further down in the document). What is removed is an explanation of why we will not add certain features such as this one (which becomes confusing, since it was added in a later version).

@ghost ghost merged commit ab766b1 into pingcap:master Jul 10, 2020
ti-srebot pushed a commit to ti-srebot/docs that referenced this pull request Jul 10, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot
Copy link
Copy Markdown
Contributor

cherry pick to release-2.1 in PR #3233

ti-srebot pushed a commit to ti-srebot/docs that referenced this pull request Jul 10, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot
Copy link
Copy Markdown
Contributor

cherry pick to release-3.0 in PR #3234

ti-srebot pushed a commit to ti-srebot/docs that referenced this pull request Jul 10, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot
Copy link
Copy Markdown
Contributor

cherry pick to release-3.1 in PR #3235

ti-srebot pushed a commit to ti-srebot/docs that referenced this pull request Jul 10, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot
Copy link
Copy Markdown
Contributor

cherry pick to release-4.0 in PR #3236

ghost pushed a commit that referenced this pull request Jul 10, 2020
* cherry pick #3192 to release-3.0

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>

* Update TOC.md

* Update assign-by-files.yml

* Delete assign-by-files.yml

* Update mysql-compatibility.md

Co-authored-by: Null not nil <67764674+nullnotnil@users.noreply.github.com>
ghost pushed a commit that referenced this pull request Jul 10, 2020
* cherry pick #3192 to release-3.1

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>

* Delete assign-by-files.yml

* Update TOC.md

* Update mysql-compatibility.md

Co-authored-by: Null not nil <67764674+nullnotnil@users.noreply.github.com>
ghost pushed a commit that referenced this pull request Jul 10, 2020
* cherry pick #3192 to release-4.0

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>

* Delete assign-by-files.yml

Co-authored-by: Null not nil <67764674+nullnotnil@users.noreply.github.com>
@yikeke yikeke added translation/doing This PR's assignee is translating this PR. and removed translation/welcome Waits for a contributor to translate this PR and create a PR to the pingcap/docs-cn repository. labels Jul 20, 2020
@TomShawn TomShawn added translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. and removed translation/doing This PR's assignee is translating this PR. labels Jul 22, 2020
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/LGT1 Indicates that a PR has LGTM 1. status/PTAL This PR is ready for reviewing. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sql-mode compatibility docs are out of date

5 participants