hughhhh commented on a change in pull request #14826: URL: https://github.com/apache/superset/pull/14826#discussion_r639218984
########## File path: docs/src/pages/docs/Connecting to Databases/mysql.mdx ########## @@ -8,16 +8,22 @@ version: 1 ## MySQL -The recommended connector library for MySQL is [mysql-connector-python](https://pypi.org/project/mysql-connector-python/). +The recommended connector library for MySQL is [mysqlclient](https://pypi.org/project/mysqlclient/). Here's the connection string: ``` -mysql+mysqlconnector://{username}:{password}@{host}/{database} +mysql://{username}:{password}@{host}/{database} ``` Host: - For Localhost or Docker running Linux: `localhost` or `127.0.0.1` - For On Prem: IP address or Host name - For Docker running in OSX: `docker.for.mac.host.internal` Port: `3306` by default + +One problem with `mysqlclient` is that it will fail to connect to newer MySQL databases using `caching_sha2_password` for authentication, since the plugin is not included in the client. In the case, you should use [mysql-connector-python](https://pypi.org/project/mysql-connector-python/) instead: Review comment: ```suggestion One problem with `mysqlclient` is that it will fail to connect to newer MySQL databases using `caching_sha2_password` for authentication, since the plugin is not included in the client. In this case, you should use `[mysql-connector-python](https://pypi.org/project/mysql-connector-python/)` instead: ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
