-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
HHH-19872: CriteriaUpdate for generic fields #11130
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
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.
Thank you @tcordel, the idea in your PR looks correct - I added a few comments on how to refine it.
hibernate-core/src/main/java/org/hibernate/query/sqm/internal/TypecheckUtil.java
Show resolved
Hide resolved
hibernate-core/src/main/java/org/hibernate/query/sqm/internal/TypecheckUtil.java
Outdated
Show resolved
Hide resolved
...ate-core/src/test/java/org/hibernate/orm/test/annotations/generics/GenericHierarchyTest.java
Outdated
Show resolved
Hide resolved
...a/org/hibernate/orm/test/annotations/generics/GenericMappedSuperclassPropertyUpdateTest.java
Show resolved
Hide resolved
Thanks for the review, changes were made according to your recommandations. |
hibernate-core/src/main/java/org/hibernate/query/sqm/internal/TypecheckUtil.java
Show resolved
Hide resolved
hibernate-core/src/main/java/org/hibernate/query/sqm/internal/TypecheckUtil.java
Outdated
Show resolved
Hide resolved
It should be OK now 🤞. Thank's ! |
50cda3b
to
78fe917
Compare
...ate-core/src/test/java/org/hibernate/orm/test/annotations/generics/GenericHierarchyTest.java
Outdated
Show resolved
Hide resolved
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.
LGTM, thanks @tcordel! I'll merge once CI passes
Generated model gen has not been refactor in test, i'll send a new patch |
Seems to be ok now |
Is there anything i can do for this pipeline error ? |
That's fine don't worry, merging |
[Please describe here what your change is about]
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.
https://hibernate.atlassian.net/browse/HHH-19872