Skip to content

Conversation

gavinking
Copy link
Member

No description provided.

CompletionStage<Object> reactiveImmediateLoad(String entityName, Object id);

CompletionStage<Void> reactiveInitializeCollection(PersistentCollection collection, boolean writing);
CompletionStage<Void> reactiveInitializeCollection(PersistentCollection<?> collection, boolean writing);

Check notice

Code scanning / CodeQL

Useless parameter

The parameter 'writing' is never used.
also remove some unnecessary operations of ReactiveQueryProducer

also fill in some missing method implementations
}

String previousFetchProfile = getLoadQueryInfluencers().getInternalFetchProfile();
final String previousFetchProfile = getLoadQueryInfluencers().getInternalFetchProfile();

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation

Invoking [LoadQueryInfluencers.getInternalFetchProfile](1) should be avoided because it has been deprecated.
@Override
public <T> CompletionStage<T> reactiveFind(Class<T> entityClass, Map<String, Object> ids) {
EntityPersister persister = getFactory().getMetamodel().locateEntityPersister( entityClass );
final EntityPersister persister = getFactory().getMetamodel().locateEntityPersister( entityClass );

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation

Invoking [MappingMetamodel.locateEntityPersister](1) should be avoided because it has been deprecated.
@Override
public <T> CompletionStage<T> reactiveFind(Class<T> entityClass, Map<String, Object> ids) {
EntityPersister persister = getFactory().getMetamodel().locateEntityPersister( entityClass );
final EntityPersister persister = getFactory().getMetamodel().locateEntityPersister( entityClass );

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation

Invoking [SessionFactoryImplementor.getMetamodel](1) should be avoided because it has been deprecated.
@gavinking gavinking merged commit 2fd2117 into hibernate:main Apr 13, 2023
@gavinking gavinking deleted the sync-sessionimpl branch June 15, 2023 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant