-
Notifications
You must be signed in to change notification settings - Fork 8k
Interoperability between Quarkus 3 and Hibernate ORM 6.x #19387
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
Conversation
This reverts commit 2f3fe2b.
…mplemented for EntityType" Related to: keycloak#19321
This avoids Hibernate 6.2.0.CR4 to fail with 'BasicValue cannot be cast to class ToOne'. It used to work on Hibernate 6.2.0.CR3.
…o create SAAJ meta-factory: Provider com.sun.xml.messaging.saaj.soap.SAAJMetaFactoryImpl not found" when running SAML.
… this may indicate a semantic (user query) problem or a bug in the parser" This behavior changed when upgrading from Hibernate 6.2.0.CR3 to 6.2.0.CR4
…ScopeTest". This behavior changed when upgrading to Hibernate 6
…om VARCHARs It treated the single byte String with its ASCII values, which then didn't find a representation in the enum's values, which lead to a "ArrayIndexOutOfBoundsException: Index 48" for a value "0" (ASCII 48). This behavior changed when migrating from Hibernate 5 to Hibernate 6. Hibernate expects a TinyInt value for all Enums by default, and this annotation overrides it.
|
@sguilhen & @vmuzikar - this PR is now reasonably green, with no failing test that I would attribute to the Hibernate 6 upgrade. I'd be happy if you could give it a review. There are the following open workarounds which are tracked separately:
@vmuzikar - I added In the Base test suite part 2, there are two adapter tests failing, which are IMHO unrelated to the Hibernate 6 upgrade:
|
sguilhen
left a comment
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.
vmuzikar
left a comment
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.
|
|
||
| <properties> | ||
| <resteasy.versions>6.0.0.Final</resteasy.versions> | ||
| <resteasy.versions>6.2.3.Final</resteasy.versions> |
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.
Just a nitpick, maybe we could stop overriding it and use the version from the root POM?
Line 109 in 8512205
| <resteasy.version>6.2.2.Final</resteasy.version> |
This PR continues the work started in #17664
It includes the changes to make Hibernate 6 which was introduces in Quarkus 3.0.0.Alpha6 work with Keycloak.