From 949adc92c23e64112129c72b2e849d9c12347da4 Mon Sep 17 00:00:00 2001 From: Brett Meyer Date: Mon, 10 Feb 2014 15:00:56 -0500 Subject: [PATCH] HHH-8945 introduced "hibernate.session.initial_capacity" to reduce StatefulPersistenceContext's Map#resize calls --- .../org/hibernate/cfg/AvailableSettings.java | 10 ++++ .../internal/StatefulPersistenceContext.java | 51 +++++++++++-------- 2 files changed, 40 insertions(+), 21 deletions(-) diff --git a/hibernate-core/src/main/java/org/hibernate/cfg/AvailableSettings.java b/hibernate-core/src/main/java/org/hibernate/cfg/AvailableSettings.java index 7aa0874c95..0ad24e8e76 100644 --- a/hibernate-core/src/main/java/org/hibernate/cfg/AvailableSettings.java +++ b/hibernate-core/src/main/java/org/hibernate/cfg/AvailableSettings.java @@ -710,5 +710,15 @@ public interface AvailableSettings { String LOG_SESSION_METRICS = "hibernate.session.events.log"; String AUTO_SESSION_EVENTS_LISTENER = "hibernate.session.events.auto"; + + /** + * A {@link org.hibernate.engine.internal.StatefulPersistenceContext} maintains multiple Maps for a Session. For + * performance reasons, it can be desirable to initialize the Maps with a non-default capacity, minimizing the + * occurrences of #resize calls. The capacity generally correlates with the quantity of entities expected to be + * managed by an average Session. + * + * Default: 8 + */ + String SESSION_INITIAL_CAPACITY = "hibernate.session.initial_capacity"; } diff --git a/hibernate-core/src/main/java/org/hibernate/engine/internal/StatefulPersistenceContext.java b/hibernate-core/src/main/java/org/hibernate/engine/internal/StatefulPersistenceContext.java index bd35eaa43d..8e0123e98d 100644 --- a/hibernate-core/src/main/java/org/hibernate/engine/internal/StatefulPersistenceContext.java +++ b/hibernate-core/src/main/java/org/hibernate/engine/internal/StatefulPersistenceContext.java @@ -51,7 +51,9 @@ import org.hibernate.action.spi.AfterTransactionCompletionProcess; import org.hibernate.cache.spi.NaturalIdCacheKey; import org.hibernate.cache.spi.access.NaturalIdRegionAccessStrategy; import org.hibernate.cache.spi.access.SoftLock; +import org.hibernate.cfg.AvailableSettings; import org.hibernate.collection.spi.PersistentCollection; +import org.hibernate.engine.config.spi.ConfigurationService; import org.hibernate.engine.loading.internal.LoadContexts; import org.hibernate.engine.spi.AssociationKey; import org.hibernate.engine.spi.BatchFetchQueue; @@ -76,7 +78,6 @@ import org.hibernate.proxy.HibernateProxy; import org.hibernate.proxy.LazyInitializer; import org.hibernate.tuple.ElementWrapper; import org.hibernate.type.CollectionType; - import org.jboss.logging.Logger; /** @@ -96,7 +97,9 @@ public class StatefulPersistenceContext implements PersistenceContext { ); private static final boolean TRACE_ENABLED = LOG.isTraceEnabled(); - private static final int INIT_COLL_SIZE = 8; + + private static final int DEFAULT_INITIAL_CAPACITY = 8; + private final int initialCapacity; private SessionImplementor session; @@ -161,20 +164,23 @@ public class StatefulPersistenceContext implements PersistenceContext { */ public StatefulPersistenceContext(SessionImplementor session) { this.session = session; + + initialCapacity = session.getFactory().getServiceRegistry().getService( ConfigurationService.class ) + .getSetting( AvailableSettings.SESSION_INITIAL_CAPACITY, Integer.class, DEFAULT_INITIAL_CAPACITY ); - entitiesByKey = new HashMap( INIT_COLL_SIZE ); - entitiesByUniqueKey = new HashMap( INIT_COLL_SIZE ); + entitiesByKey = new HashMap( initialCapacity ); + entitiesByUniqueKey = new HashMap( initialCapacity ); //noinspection unchecked - proxiesByKey = new ConcurrentReferenceHashMap( INIT_COLL_SIZE, .75f, 1, ConcurrentReferenceHashMap.ReferenceType.STRONG, ConcurrentReferenceHashMap.ReferenceType.WEAK, null ); - entitySnapshotsByKey = new HashMap( INIT_COLL_SIZE ); + proxiesByKey = new ConcurrentReferenceHashMap( initialCapacity, .75f, 1, ConcurrentReferenceHashMap.ReferenceType.STRONG, ConcurrentReferenceHashMap.ReferenceType.WEAK, null ); + entitySnapshotsByKey = new HashMap( initialCapacity ); entityEntryContext = new EntityEntryContext(); -// entityEntries = IdentityMap.instantiateSequenced( INIT_COLL_SIZE ); - collectionEntries = IdentityMap.instantiateSequenced( INIT_COLL_SIZE ); - parentsByChild = new IdentityHashMap( INIT_COLL_SIZE ); +// entityEntries = IdentityMap.instantiateSequenced( initialCapacity ); + collectionEntries = IdentityMap.instantiateSequenced( initialCapacity ); + parentsByChild = new IdentityHashMap( initialCapacity ); - collectionsByKey = new HashMap( INIT_COLL_SIZE ); - arrayHolders = new IdentityHashMap( INIT_COLL_SIZE ); + collectionsByKey = new HashMap( initialCapacity ); + arrayHolders = new IdentityHashMap( initialCapacity ); nullifiableEntityKeys = new HashSet(); @@ -182,8 +188,8 @@ public class StatefulPersistenceContext implements PersistenceContext { } private void initTransientState() { - nullAssociations = new HashSet( INIT_COLL_SIZE ); - nonlazyCollections = new ArrayList( INIT_COLL_SIZE ); + nullAssociations = new HashSet( initialCapacity ); + nonlazyCollections = new ArrayList( initialCapacity ); } @Override @@ -207,7 +213,7 @@ public class StatefulPersistenceContext implements PersistenceContext { @Override public void addUnownedCollection(CollectionKey key, PersistentCollection collection) { if (unownedCollections==null) { - unownedCollections = new HashMap(INIT_COLL_SIZE); + unownedCollections = new HashMap(initialCapacity); } unownedCollections.put( key, collection ); } @@ -1441,6 +1447,9 @@ public class StatefulPersistenceContext implements PersistenceContext { } final StatefulPersistenceContext rtn = new StatefulPersistenceContext( session ); SessionFactoryImplementor sfi = session.getFactory(); + + int initialCapacity = sfi.getServiceRegistry().getService( ConfigurationService.class ) + .getSetting( AvailableSettings.SESSION_INITIAL_CAPACITY, Integer.class, DEFAULT_INITIAL_CAPACITY ); // during deserialization, we need to reconnect all proxies and // collections to this session, as well as the EntityEntry and @@ -1456,7 +1465,7 @@ public class StatefulPersistenceContext implements PersistenceContext { if ( tracing ) { LOG.trace( "Starting deserialization of [" + count + "] entitiesByKey entries" ); } - rtn.entitiesByKey = new HashMap( count < INIT_COLL_SIZE ? INIT_COLL_SIZE : count ); + rtn.entitiesByKey = new HashMap( count < initialCapacity ? initialCapacity : count ); for ( int i = 0; i < count; i++ ) { rtn.entitiesByKey.put( EntityKey.deserialize( ois, sfi ), ois.readObject() ); } @@ -1465,7 +1474,7 @@ public class StatefulPersistenceContext implements PersistenceContext { if ( tracing ) { LOG.trace( "Starting deserialization of [" + count + "] entitiesByUniqueKey entries" ); } - rtn.entitiesByUniqueKey = new HashMap( count < INIT_COLL_SIZE ? INIT_COLL_SIZE : count ); + rtn.entitiesByUniqueKey = new HashMap( count < initialCapacity ? initialCapacity : count ); for ( int i = 0; i < count; i++ ) { rtn.entitiesByUniqueKey.put( EntityUniqueKey.deserialize( ois, session ), ois.readObject() ); } @@ -1476,7 +1485,7 @@ public class StatefulPersistenceContext implements PersistenceContext { } //noinspection unchecked rtn.proxiesByKey = new ConcurrentReferenceHashMap( - count < INIT_COLL_SIZE ? INIT_COLL_SIZE : count, + count < initialCapacity ? initialCapacity : count, .75f, 1, ConcurrentReferenceHashMap.ReferenceType.STRONG, @@ -1502,7 +1511,7 @@ public class StatefulPersistenceContext implements PersistenceContext { if ( tracing ) { LOG.trace( "Starting deserialization of [" + count + "] entitySnapshotsByKey entries" ); } - rtn.entitySnapshotsByKey = new HashMap( count < INIT_COLL_SIZE ? INIT_COLL_SIZE : count ); + rtn.entitySnapshotsByKey = new HashMap( count < initialCapacity ? initialCapacity : count ); for ( int i = 0; i < count; i++ ) { rtn.entitySnapshotsByKey.put( EntityKey.deserialize( ois, sfi ), ois.readObject() ); } @@ -1513,7 +1522,7 @@ public class StatefulPersistenceContext implements PersistenceContext { if ( tracing ) { LOG.trace( "Starting deserialization of [" + count + "] collectionsByKey entries" ); } - rtn.collectionsByKey = new HashMap( count < INIT_COLL_SIZE ? INIT_COLL_SIZE : count ); + rtn.collectionsByKey = new HashMap( count < initialCapacity ? initialCapacity : count ); for ( int i = 0; i < count; i++ ) { rtn.collectionsByKey.put( CollectionKey.deserialize( ois, session ), (PersistentCollection) ois.readObject() ); } @@ -1522,7 +1531,7 @@ public class StatefulPersistenceContext implements PersistenceContext { if ( tracing ) { LOG.trace( "Starting deserialization of [" + count + "] collectionEntries entries" ); } - rtn.collectionEntries = IdentityMap.instantiateSequenced( count < INIT_COLL_SIZE ? INIT_COLL_SIZE : count ); + rtn.collectionEntries = IdentityMap.instantiateSequenced( count < initialCapacity ? initialCapacity : count ); for ( int i = 0; i < count; i++ ) { final PersistentCollection pc = (PersistentCollection) ois.readObject(); final CollectionEntry ce = CollectionEntry.deserialize( ois, session ); @@ -1534,7 +1543,7 @@ public class StatefulPersistenceContext implements PersistenceContext { if ( tracing ) { LOG.trace( "Starting deserialization of [" + count + "] arrayHolders entries" ); } - rtn.arrayHolders = new IdentityHashMap( count < INIT_COLL_SIZE ? INIT_COLL_SIZE : count ); + rtn.arrayHolders = new IdentityHashMap( count < initialCapacity ? initialCapacity : count ); for ( int i = 0; i < count; i++ ) { rtn.arrayHolders.put( ois.readObject(), (PersistentCollection) ois.readObject() ); }