-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Add quarkus.hibernate-orm.database.version-check.enabled #43762
Conversation
/cc @gsmet (hibernate-orm) |
Marking for backport, as this workaround is needed to make Quarkus 3.15 work with CockroachDB in particular. |
79d8b0c
to
426f839
Compare
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks nice 🎉
426f839
to
65fc626
Compare
This comment has been minimized.
This comment has been minimized.
This allows disabling the check on startup if one knows the database won't be reachable. It also currently defaults to being disabled when a dialect is set explicitly (`quarkus.hibernate-orm.dialect=something`), in order to work around problems we have with correctly detecting the version on some databases that we don't have tests for (unsupported ones).
65fc626
to
6daa2ea
Compare
Status for workflow
|
🙈 The PR is closed and the preview is expired. |
This allows disabling the check on startup if one knows the database won't be reachable.
It also currently defaults to being disabled when a dialect is set explicitly (
quarkus.hibernate-orm.dialect=something
), in order to work around problems we have with correctly detecting the version on some databases that we don't have tests for (unsupported ones).This is a workaround for #43703 and #42255, but is not a fix: we will need Hibernate ORM 7.0 for an actual fix that makes database version checks work correctly even on Azure SQL Server or CockroachDB.