-
Notifications
You must be signed in to change notification settings - Fork 359
MariadbDialect needs an INSTANCE property. #2060
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
Comments
I'm not convinced a singleton instance makes sense for the From https://mariadb.com/kb/en/identifier-names :
From https://mariadb.com/kb/en/identifier-case-sensitivity/ :
Regarding
The "correct" is to use the Is there something else I'm missing? |
Aren't most of those things true for MySQL too? MySQL and MariaDB are pretty much the same code base. MySQL has a singleton instance. |
That is correct, but would be more of an argument for removing that |
Uh oh!
There was an error while loading. Please reload this page.
MariaDbDialect
extendsMySqlDialect
, but does not implement it's own public staticINSTANCE
that returns aMariaDbDialect
instance. This makesMariaDbDialect.INSTANCE
indistinguishable fromMySqlDialect.INSTANCE
usinginstanceof
.We can create it manually, i.e.
new MariaDbDialect(MySqlDialect.MYSQL_IDENTIFIER_PROCESSING)
But it would be nice for
MariaDbDialect
to have that out of the box. If using a starter, then it would be problematic to create the dialect "the right way".The text was updated successfully, but these errors were encountered: