Skip to content

dev, v2.1, v3.0/reference: change the description of SQL_MODE#1776

Merged
lilin90 merged 3 commits into
pingcap:masterfrom
wshwsh12:sql_mode
Aug 22, 2019
Merged

dev, v2.1, v3.0/reference: change the description of SQL_MODE#1776
lilin90 merged 3 commits into
pingcap:masterfrom
wshwsh12:sql_mode

Conversation

@wshwsh12
Copy link
Copy Markdown
Contributor

@wshwsh12 wshwsh12 commented Aug 19, 2019

What is changed, added or deleted?

Fix the description of SQL_MODE.
Now TiDB default SQL mode is same with MySQL 5.7
And TiDB does currently enable the ONLY_FULL_GROUP_BY mode by default.

Which version does your change affect?

dev, v3.0, v2.1

Closed the issue: #1742

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Aug 19, 2019

CLA assistant check
All committers have signed the CLA.

@wshwsh12 wshwsh12 requested a review from lilin90 August 19, 2019 11:41
@lilin90 lilin90 requested a review from TomShawn August 21, 2019 02:42
@lilin90 lilin90 added the translation/doing This PR’s assignee is translating this PR. label Aug 21, 2019
Comment thread dev/reference/mysql-compatibility.md Outdated
+ 默认 SQL mode 不同:
+ TiDB 中为 `STRICT_TRANS_TABLES,NO_ENGINE_SUBSTITUTION`
+ MySQL 中为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
+ 默认 SQL mode **已相同**
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
+ 默认 SQL mode **已相同**
+ 默认 SQL mode 与 MySQL **已相同**

Comment thread v2.1/reference/mysql-compatibility.md Outdated
+ 默认 SQL mode 不同:
+ TiDB 中为 `STRICT_TRANS_TABLES,NO_ENGINE_SUBSTITUTION`
+ MySQL 中为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
+ 默认 SQL mode **已相同**
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
+ 默认 SQL mode **已相同**
+ 默认 SQL mode 与 MySQL **已相同**

Comment thread v3.0/reference/mysql-compatibility.md Outdated
+ 默认 SQL mode 不同:
+ TiDB 中为 `STRICT_TRANS_TABLES,NO_ENGINE_SUBSTITUTION`
+ MySQL 中为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
+ 默认 SQL mode **已相同**
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
+ 默认 SQL mode **已相同**
+ 默认 SQL mode 与 MySQL **已相同**

Comment thread dev/reference/mysql-compatibility.md Outdated
+ TiDB 中为 `STRICT_TRANS_TABLES,NO_ENGINE_SUBSTITUTION`
+ MySQL 中为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
+ 默认 SQL mode **已相同**
+ TiDB 和 MySQL 5.7 均为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
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
+ TiDB 和 MySQL 5.7 均为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
+ TiDB 和 MySQL 5.7 中均为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`

Comment thread v2.1/reference/mysql-compatibility.md Outdated
+ TiDB 中为 `STRICT_TRANS_TABLES,NO_ENGINE_SUBSTITUTION`
+ MySQL 中为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
+ 默认 SQL mode **已相同**
+ TiDB 和 MySQL 5.7 均为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
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
+ TiDB 和 MySQL 5.7 均为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
+ TiDB 和 MySQL 5.7 中均为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`

Comment thread v3.0/reference/mysql-compatibility.md Outdated
+ TiDB 中为 `STRICT_TRANS_TABLES,NO_ENGINE_SUBSTITUTION`
+ MySQL 中为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
+ 默认 SQL mode **已相同**
+ TiDB 和 MySQL 5.7 均为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
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
+ TiDB 和 MySQL 5.7 均为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
+ TiDB 和 MySQL 5.7 中均为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`

@TomShawn TomShawn changed the title Fix the description of SQL_MODE. dev, v2.1, v3.0/referece: change the description of SQL_MODE Aug 21, 2019
@TomShawn TomShawn changed the title dev, v2.1, v3.0/referece: change the description of SQL_MODE dev, v2.1, v3.0/reference: change the description of SQL_MODE Aug 21, 2019
@wshwsh12
Copy link
Copy Markdown
Contributor Author

@TomShawn PTAL again.

Copy link
Copy Markdown
Contributor

@TomShawn TomShawn left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@lilin90 lilin90 left a comment

Choose a reason for hiding this comment

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

LGTM

@wshwsh12 wshwsh12 requested a review from XuHuaiyu August 21, 2019 09:38
Copy link
Copy Markdown
Contributor

@XuHuaiyu XuHuaiyu left a comment

Choose a reason for hiding this comment

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

LGTM

+ TiDB 中为 `STRICT_TRANS_TABLES,NO_ENGINE_SUBSTITUTION`
+ MySQL 中为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
+ 默认 SQL mode 与 MySQL **已相同**
+ TiDB 和 MySQL 5.7 中均为 `ONLY_FULL_GROUP_BY,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION`
Copy link
Copy Markdown
Contributor

@junlan-zhang junlan-zhang Sep 29, 2019

Choose a reason for hiding this comment

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

@wshwsh12 Should we remove it from 默认设置的区别 part because there is no difference? We can still refer to SQL模式 for the default value of SQL mode of TiDB.

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.

This part has been updated in #2107.

@junlan-zhang junlan-zhang 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 Sep 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

6 participants