MySQL sql_mode change

[harikrishna-patnala]

MySQL sql_mode has changed and if the admin uses MySQL directly and wants to query tables it is advised to change the sql_mode in the corresponding session or globally.

Eg. mysql> set global sql_mode="STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_ENGINE_SUBSTITUTION";
Query OK, 0 rows affected (0.00 sec)

[harikrishna-patnala]

This addresses the issue apache/cloudstack#5175

[harikrishna-patnala]

@blueorangutan docbuild

[blueorangutan]

@harikrishna-patnala a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

[blueorangutan]

Doc build preview: http://qa.cloudstack.cloud/docs/WIP-PROOFING/pr/239. (SL-JID 137)

[nvazquez]

Thanks @harikrishna-patnala, content LGTM but needs formatting

[harikrishna-patnala]

Thanks @nvazquez , fixed formatting

[harikrishna-patnala]

@blueorangutan docbuild

[blueorangutan]

Doc build preview: http://qa.cloudstack.cloud/docs/WIP-PROOFING/pr/239. (SL-JID 138)

[blueorangutan]

@harikrishna-patnala a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

[blueorangutan]

Doc build preview: http://qa.cloudstack.cloud/docs/WIP-PROOFING/pr/239. (SL-JID 139)

[nvazquez]

LGTM

[weizhouapache]

@harikrishna-patnala
would it be better to add few lines for persistent mysql setting ?
users need to change /etc/mysql/my.cnfand restart mysql

[yadvr]

+1

@harikrishna-patnala I think we should advise admin to just fix the global my.cnf? See install/admin section?

[kricud]

@rhtyd @weizhouapache Unpopular opinion: I would not decrease strictness of sql globally if its absolutely not needed.
Someone has com to conclusion that it must be stricter globally and there probably are arguments why in context of
enabling ONLY_FULL_GROUP_BY. Approach of allowing it to each client/connection in my opinion is better.
Mentioning it in documentation in my opinion is enough.
https://www.percona.com/blog/2019/05/13/solve-query-failures-regarding-only_full_group_by-sql-mode/

[yadvr]

I suppose with your comment @kricud, the changes can be accepted then.

[yadvr]

PR merged cc @GabrielBrascher