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

feat: added support to use mysql jdbc driver #126

Merged
merged 2 commits into from
Jun 19, 2024

Conversation

githubjianli
Copy link
Collaborator

@githubjianli githubjianli commented Jun 18, 2024

📝 Description

  • Added MYSQL_CONNECTION_DRIVER_NAME to support use different connection driver, defaults: com.mysql.jdbc.Driver.
  • Added MYSQL_TYPE to support use different type of MySQL, defaults: mysql.
  • Added mysql-connector-java to support to use driver com.mysql.jdbc.Driver.

🔗 Related Issues

@githubjianli githubjianli requested a review from a team as a code owner June 18, 2024 21:42
mroark1m
mroark1m previously approved these changes Jun 18, 2024
@@ -3,6 +3,12 @@ All notable changes to this project will be documented in this file.

The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).

## [4.1.0] - 2024-06-19
### Added
- Added `MYSQL_CONNECTION_DRIVER_NAME` to support use different connection driver, defaults: `com.mysql.jdbc.Driver`.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be a major version release? Changing the driver from Maria to MySQL could be a breaking change.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

they are actually same, but we just trying to make it make better sense, so mariadb is using mariabdb connector jar, mysql is using mysql jar, so I think this is a minor change.

Copy link
Contributor

@patduin patduin Jun 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't someone using MariaDB RDS moving to this version now suddenly using the mysql driver? Maybe a major version makes sense here.

@javsanbel2 javsanbel2 merged commit ea075d8 into master Jun 19, 2024
1 check passed
@javsanbel2 javsanbel2 deleted the feature/add_mysql_driver branch June 19, 2024 08:32
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.

6 participants