Hibernate
/
hibernate-orm
mirror of https://github.com/hibernate/hibernate-orm
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

HHH-10989 - Always resolve CollectionType on load so that unfetched collections have a reference stored in StatefulPersistentContext 

HHH-10989 : Shorten test table names
This commit is contained in:
Gail Badner 2017-02-15 17:31:27 -08:00 committed by Andrea Boriero
parent 077ebbc04f
commit 55f7f71e04
2 changed files with 17 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
hibernate-core/src/main/java/org/hibernate/engine/internal/TwoPhaseLoad.java
View File

@ -34,7 +34,6 @@ import org.hibernate.persister.entity.EntityPersister;
import org.hibernate.pretty.MessageHelper; import org.hibernate.pretty.MessageHelper;
import org.hibernate.property.access.internal.PropertyAccessStrategyBackRefImpl; import org.hibernate.property.access.internal.PropertyAccessStrategyBackRefImpl;
import org.hibernate.proxy.HibernateProxy; import org.hibernate.proxy.HibernateProxy;
import org.hibernate.type.CollectionType;
import org.hibernate.type.Type; import org.hibernate.type.Type;
import org.hibernate.type.TypeHelper; import org.hibernate.type.TypeHelper;
@ -148,15 +147,24 @@ public final class TwoPhaseLoad {
final Type[] types = persister.getPropertyTypes(); final Type[] types = persister.getPropertyTypes();
for ( int i = 0; i < hydratedState.length; i++ ) { for ( int i = 0; i < hydratedState.length; i++ ) {
final Object value = hydratedState[i]; final Object value = hydratedState[i];
if ( value!=LazyPropertyInitializer.UNFETCHED_PROPERTY && value!= PropertyAccessStrategyBackRefImpl.UNKNOWN ) { if ( value == LazyPropertyInitializer.UNFETCHED_PROPERTY ) {
hydratedState[i] = types[i].resolve( value, session, entity ); // IMPLEMENTATION NOTE: This is a lazy property on a bytecode-enhanced entity.
} // hydratedState[i] needs to remain LazyPropertyInitializer.UNFETCHED_PROPERTY so that
else if ( types[i].isCollectionType() ) { // setPropertyValues() below (ultimately AbstractEntityTuplizer#setPropertyValues) works properly
// HHH-10989 Even if not fetched, resolve a collection so that a CollectionReference is added to StatefulPersistentContext // No resolution is necessary, unless the lazy property is a collection.
// No assignment to the hydratedState, that would trigger the load of the collection (below, on setPropertyValues) if ( types[i].isCollectionType() ) {
// IMPLEMENTATION NOTE: this is a lazy collection property on a bytecode-enhanced entity.
// HHH-10989: We need to resolve the collection so that a CollectionReference is added to StatefulPersistentContext.
// As mentioned above, hydratedState[i] needs to remain LazyPropertyInitializer.UNFETCHED_PROPERTY
// so do not assign the resolved, unitialized PersistentCollection back to hydratedState[i].
types[i].resolve( value, session, entity ); types[i].resolve( value, session, entity );
} }
} }
else if ( value!= PropertyAccessStrategyBackRefImpl.UNKNOWN ) {
// we know value != LazyPropertyInitializer.UNFETCHED_PROPERTY
hydratedState[i] = types[i].resolve( value, session, entity );
}
}
//Must occur afterQuery resolving identifiers! //Must occur afterQuery resolving identifiers!
if ( session.isEventSource() ) { if ( session.isEventSource() ) {

4
hibernate-core/src/test/java/org/hibernate/test/bytecode/enhancement/cascade/CascadeWithFkConstraintTestTask.java
View File

@ -100,7 +100,7 @@ public class CascadeWithFkConstraintTestTask extends AbstractEnhancerTestTask {
// --- // // --- //
@Entity @Entity(name="Garage")
public static class Garage { public static class Garage {
@Id @Id
@ -120,7 +120,7 @@ public class CascadeWithFkConstraintTestTask extends AbstractEnhancerTestTask {
} }
@Entity @Entity(name="Car")
public static class Car { public static class Car {
@Id @Id