-
Notifications
You must be signed in to change notification settings - Fork 3.6k
HHH-19364 changes to Specifications to support Hibernate Processor #10066
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
2b84dd9
to
fa0c470
Compare
1. fix generic types of parameters (should be lower bounds) 2. make it work with plain-vanilla EntityManager 3. to eliminate ambiguity in overload resolution, this unfortunately required disallowing SharedSessionContract 4. make Path.fetch() use LEFT join
fa0c470
to
876018d
Compare
cc @sebersole |
|
||
/** | ||
* Finalize the building and create executable query instance. | ||
*/ | ||
CommonQueryContract createQuery(SharedSessionContract session); | ||
CommonQueryContract createQuery(EntityManager entityManager); |
Check notice
Code scanning / CodeQL
Confusing overloading of methods Note
createQuery
@@ -101,6 +113,11 @@ | |||
return new QuerySqmImpl<>( sqmStatement, true, null, sessionImpl ); | |||
} | |||
|
|||
@Override | |||
public MutationQuery createQuery(EntityManager entityManager) { |
Check notice
Code scanning / CodeQL
Confusing overloading of methods Note
createQuery
@@ -145,6 +157,11 @@ | |||
return new SqmSelectionQueryImpl<>( sqmStatement, true, resultType, sessionImpl ); | |||
} | |||
|
|||
@Override | |||
public SelectionQuery<T> createQuery(EntityManager entityManager) { |
Check notice
Code scanning / CodeQL
Confusing overloading of methods Note
EntityManager
SharedSessionContract
Path.fetch()
useLEFT
join[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-19364