-
Notifications
You must be signed in to change notification settings - Fork 1k
Detect MariaDB vs MySQL #6072
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
Detect MariaDB vs MySQL #6072
Conversation
|
@mrsdizzie @schlessera curious to hear your thoughts |
Codecov ReportAttention: Patch coverage is
📢 Thoughts on this report? Let us know! |
|
Maybe it could be good to have one of the test runs use Mariadb instead of MySQL as well? |
|
Yes, see wp-cli/.github#123 Feel free to submit a PR :) |
Because we need these both in this project and in wp-cli-tests
|
It seems like this change may have introduced a hiccup in our workflow with wp-cli and mariadb. in my wordpress docker container i'm running Debug (db): Final MySQL command: /usr/bin/env mariadb --no-defaults --no-auto-rehash --batch --skip-column-names --host='db' --user='wordpress_user' --default-character-set='utf8' --execute='SELECT @@SESSION.sql_mode' (0.061s) I dug a little deeper to find a little more debugging info. running the command diretly shows the same error i read adding the arg from there, it does work if i also add the "--password=blah" arg. Has anyone else overcome this in their workflow with docker+mariadb+wp-cli and can offer help? |
See:
MariaDB currently adds symlinks for
mysqlto usemariadbunder the hood, but that is being deprecated and people get warnings likeDeprecated program name. It will be removed in a future release, use 'mariadb' insteadWith this change, the idea is to offer better utils for commands like db-command to pick the right binary.
I suppose that could imply that we officially support MariaDB, but I guess since WordPress core itself recommends it, we should at least test it.