-
Notifications
You must be signed in to change notification settings - Fork 95
Upgrade Hibernate ORM to 7.0.0.CR1 #2221
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
instead of EventSource
now that refactoring to core haa made this possible
factory.getServiceRegistry().requireService( BatchLoaderFactory.class ) | ||
.createEntityBatchLoader( domainBatchSize, entityDescriptor, factory ); |
Check notice
Code scanning / CodeQL
Deprecated method or constructor invocation Note
BatchLoaderFactory.createEntityBatchLoader
list.add( pos < 0 ? null : resultList.get(pos) ); | ||
return getEntityPersister( entityClass.getName() ) | ||
.reactiveMultiLoad( sids, this, StatelessSessionImpl.MULTI_ID_LOAD_OPTIONS ) | ||
.whenComplete( (v, e) -> { |
Check notice
Code scanning / CodeQL
Useless parameter Note
list.add( pos < 0 ? null : resultList.get(pos) ); | ||
return getEntityPersister( entityClass.getName() ) | ||
.reactiveMultiLoad( sids, this, StatelessSessionImpl.MULTI_ID_LOAD_OPTIONS ) | ||
.whenComplete( (v, e) -> { |
Check notice
Code scanning / CodeQL
Useless parameter Note
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.
All LGTM!
Plus, update expected Oracle default database mapping in tests: * `Double` and `double` java types changes from `FLOAT` to `BINARY_DOUBLE` * `Float` and `float` java types changes from `FLOAT` to `BINARY_FLOAT` Changes to ReactiveAbstractSelectionQuery
Fix #2220
Fix #2214
If CI is happy, I think we can merge this now