[GitHub] [hive] zabetak commented on a diff in pull request #3250: HIVE-26180: Change MySQLConnectorProvider driver from mariadb to mysql

[GitBox <gi...@apache.org>]

zabetak commented on code in PR #3250:
URL: https://github.com/apache/hive/pull/3250#discussion_r860067577


##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/dataconnector/jdbc/MySQLConnectorProvider.java:
##########
@@ -31,7 +31,7 @@
 public class MySQLConnectorProvider extends AbstractJDBCConnectorProvider {
   private static Logger LOG = LoggerFactory.getLogger(MySQLConnectorProvider.class);
 
-  private static final String DRIVER_CLASS = "org.mariadb.jdbc.Driver";
+  private static final String DRIVER_CLASS = "com.mysql.jdbc.Driver";

Review Comment:
   I am not sure why HIVE-25423 changed the driver to use mariadb but maybe it has to do with the fact that the added test (dataconnector_mysql.q) was also using a docker image of MariaDB (even though the respective class was implying that it was MySQL) before I fix it on HIVE-25594.



-- 
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.

To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org