-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
HHH-17377 - Migrate to JPA 3.2 #7494
Conversation
The second commit fixes SQM/JPA Criteria compile errors. Just Graph compile errors left - only 32! |
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 good so far, I just left a few comments from a quick glance over the code so we don't forget anything. One particular change in the JPA spec regarding createNativeQuery
is a bit unfortunate and I hope we can improve the spec API.
hibernate-core/src/main/java/org/hibernate/engine/spi/SessionDelegatorBaseImpl.java
Outdated
Show resolved
Hide resolved
...te-core/src/main/java/org/hibernate/query/criteria/spi/HibernateCriteriaBuilderDelegate.java
Show resolved
Hide resolved
...te-core/src/main/java/org/hibernate/query/criteria/spi/HibernateCriteriaBuilderDelegate.java
Outdated
Show resolved
Hide resolved
hibernate-core/src/main/java/org/hibernate/query/sqm/internal/SqmCriteriaNodeBuilder.java
Show resolved
Hide resolved
hibernate-core/src/main/java/org/hibernate/query/sqm/internal/SqmCriteriaNodeBuilder.java
Outdated
Show resolved
Hide resolved
hibernate-core/src/main/java/org/hibernate/query/sqm/tree/insert/SqmInsertValuesStatement.java
Outdated
Show resolved
Hide resolved
hibernate-core/src/main/java/org/hibernate/query/sqm/tree/update/SqmUpdateStatement.java
Outdated
Show resolved
Hide resolved
hibernate-core/src/main/java/org/hibernate/query/sqm/tree/update/SqmUpdateStatement.java
Show resolved
Hide resolved
hibernate-core/src/main/java/org/hibernate/query/sqm/tree/select/SqmSortSpecification.java
Outdated
Show resolved
Hide resolved
hibernate-core/src/main/java/org/hibernate/query/spi/QueryProducerImplementor.java
Outdated
Show resolved
Hide resolved
Thanks for the review @beikov. You caught some tricky ones to track down. I left a few replies |
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.
CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.
07858f4
to
bc4ab76
Compare
b3aaecb
to
c4a70f6
Compare
https://hibernate.atlassian.net/browse/HHH-17377 Fixed SQM/JPA Criteria compilation errors. Just Graph left
https://hibernate.atlassian.net/browse/HHH-17377 Just Graph completed - 2 left related to Session/EntityManager
https://hibernate.atlassian.net/browse/HHH-17377 Now on to 73 test compile errors :)
https://hibernate.atlassian.net/browse/HHH-17377 Test compilation success!! 95 test failures next
https://hibernate.atlassian.net/browse/HHH-17377 Test compilation success!! 95 test failures next
https://hibernate.atlassian.net/browse/HHH-17377 Test compilation success!! 95 test failures next
https://hibernate.atlassian.net/browse/HHH-17377 Test compilation success!! 95 test failures next
https://hibernate.atlassian.net/browse/HHH-17377 Test compilation success!! 95 test failures next
https://hibernate.atlassian.net/browse/HHH-17377 Change GitHub workflow to use Java 17
https://hibernate.atlassian.net/browse/HHH-17377 Change GitHub workflow to use Java 17
https://hibernate.atlassian.net/browse/HHH-17377 fixed check failures
https://hibernate.atlassian.net/browse/HHH-17377 XJB changes (JAXB "binding model")
https://hibernate.atlassian.net/browse/HHH-17377 XJB changes (JAXB "binding model")
https://hibernate.atlassian.net/browse/HHH-17377 XJB changes (JAXB "binding model")
https://hibernate.atlassian.net/browse/HHH-17377 XJB changes (JAXB "binding model")
https://hibernate.atlassian.net/browse/HHH-17377
https://hibernate.atlassian.net/browse/HHH-17441