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

improve mysql my.cnf settings file #4994

Closed
wants to merge 9 commits into from

Conversation

morgo
Copy link
Contributor

@morgo morgo commented Jul 15, 2019

This addresses some of the easiest suggestions in #4990.

Because this config file is used by multiple versions of MySQL, not all changes are possible. After #4987 merges, the settings only required by MySQL 5.6 (such as sql_mode) can be moved to a separate file and further cleanup is possible.

I will add future improvements in followup PRs, after splitting config file per version.

Signed-off-by: Morgan Tocker tocker@gmail.com

Partially addresses vitessio#4990. Needs separate config files per
MySQL version for additional enhancements.

Signed-off-by: Morgan Tocker <tocker@gmail.com>
@morgo morgo requested a review from sougou as a code owner July 15, 2019 01:32
Signed-off-by: Morgan Tocker <tocker@gmail.com>
CI tests are taking too long and failing

Signed-off-by: Morgan Tocker <tocker@gmail.com>
8.0 does not like no_auto_create_user

Signed-off-by: Morgan Tocker <tocker@gmail.com>
Signed-off-by: Morgan Tocker <tocker@gmail.com>
Signed-off-by: Morgan Tocker <tocker@gmail.com>
… into morgo-improve-configs

Signed-off-by: Morgan Tocker <tocker@gmail.com>
Signed-off-by: Morgan Tocker <tocker@gmail.com>
@morgo
Copy link
Contributor Author

morgo commented Jul 15, 2019

I had trouble figuring out how to signoff an earlier commit. I am going to close this in favor of #4999

@morgo morgo closed this Jul 15, 2019
@morgo morgo deleted the morgo-improve-configs branch July 15, 2019 19:14
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.

1 participant