Support for MariaDB

[happenedIn]

Would like to add support for MariaDB datasources

[hsluoyz]

@tangyang9464 @imp2002

[hsluoyz]

@happenedIn does the existing code not work for MariaDB? Why add data source? MariaDB and MySQL should share the same API. We need a justification that we have to add MariaDB datasources, especially the datasources, what's datasources?

[happenedIn]

@hsluoyz no because the existing code determines which database to perform operations using databaseProductName which is based off of the connection properties.

1. If a driver along with credentials and database url is supplied to the constructor the databaseProductName is set if the driver string contains correct database type

2. If a DataSource object is supplied to the constructor (which is what I'm using) the databaseProductName is determined from initSessionFactory() method line 160 this.databaseProductName = conn.getMetaData().getDatabaseProductName();. Since I'm connected to a MariaDB database that call returns MariaDB and this database string is not checked for in the createDatabase(), createTable(), or dropTable() methods.

I'm using a spring mvc application deployed in tomcat which uses a DataSource factory method. Something like this https://www.baeldung.com/spring-boot-configure-data-source-programmatic#configuring-a-datasource-bean

So I need to rely on the HibernateAdapter(Datasource datasource) constructor

[hsluoyz]

🎉 This issue has been resolved in version 1.4.0 🎉

The release is available on:

• GitHub release
• v1.4.0

Your semantic-release bot 📦🚀