Hibernate/hibernate-orm
1
0
Fork 0
mirror of https://github.com/hibernate/hibernate-orm synced 2025-02-28 23:09:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

HHH-13587 Introduce SPI method PersistenceContext#getCollectionEntriesSize()

Browse Source
This commit is contained in:
Sanne Grinovero 2019-08-17 17:00:13 +01:00 committed by Sanne Grinovero
parent 0d10174c23
commit 3b30c60ce3
7 changed files with 25 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
hibernate-core/src/main/java/org/hibernate/engine/internal/StatefulPersistenceContext.java
View File

@ -1768,6 +1768,11 @@ public boolean isNullifiableEntityKeysEmpty() {
return ( nullifiableEntityKeys == null || nullifiableEntityKeys.size() == 0 );
}
@Override
public int getCollectionEntriesSize() {
return collectionEntries == null ? 0 : collectionEntries.size();
}
private void cleanUpInsertedKeysAfterTransaction() {
if ( insertedKeysMap != null ) {
insertedKeysMap.clear();

7
hibernate-core/src/main/java/org/hibernate/engine/spi/PersistenceContext.java
View File

@ -745,6 +745,13 @@ CollectionEntry addInitializedCollection(CollectionPersister persister,
*/
boolean isNullifiableEntityKeysEmpty();
/**
* The size of the internal map storing all collection entries.
* (The map is not exposed directly, but the size is often useful)
* @return the size
*/
int getCollectionEntriesSize();
/**
* Provides centralized access to natural-id-related functionality.
*/

2
hibernate-core/src/main/java/org/hibernate/event/internal/AbstractFlushingEventListener.java
View File

@ -125,7 +125,7 @@ private void logFlushResults(FlushEvent event) {
session.getActionQueue().numberOfCollectionCreations(),
session.getActionQueue().numberOfCollectionUpdates(),
session.getActionQueue().numberOfCollectionRemovals(),
persistenceContext.getCollectionEntries().size()
persistenceContext.getCollectionEntriesSize()
);
new EntityPrinter( session.getFactory() ).toString(
persistenceContext.getEntitiesByKey().entrySet()

2
hibernate-core/src/main/java/org/hibernate/event/internal/DefaultAutoFlushEventListener.java
View File

@ -89,6 +89,6 @@ private boolean flushMightBeNeeded(final EventSource source) {
return !source.getHibernateFlushMode().lessThan( FlushMode.AUTO )
&& source.getDontFlushFromFind() == 0
&& ( persistenceContext.getNumberOfManagedEntities() > 0 ||
persistenceContext.getCollectionEntries().size() > 0 );
persistenceContext.getCollectionEntriesSize() > 0 );
}
}

2
hibernate-core/src/main/java/org/hibernate/event/internal/DefaultFlushEventListener.java
View File

@ -31,7 +31,7 @@ public void onFlush(FlushEvent event) throws HibernateException {
final PersistenceContext persistenceContext = source.getPersistenceContextInternal();
if ( persistenceContext.getNumberOfManagedEntities() > 0 ||
persistenceContext.getCollectionEntries().size() > 0 ) {
persistenceContext.getCollectionEntriesSize() > 0 ) {
try {
source.getEventListenerManager().flushStart();

2
hibernate-core/src/main/java/org/hibernate/stat/internal/SessionStatisticsImpl.java
View File

@ -28,7 +28,7 @@ public int getEntityCount() {
}
public int getCollectionCount() {
return session.getPersistenceContextInternal().getCollectionEntries().size();
return session.getPersistenceContextInternal().getCollectionEntriesSize();
}
public Set getEntityKeys() {

18
hibernate-core/src/test/java/org/hibernate/test/flush/TestAutoFlushBeforeQueryExecution.java
View File

@ -60,7 +60,7 @@ public void testAutoflushIsRequired() {
final PersistenceContext persistenceContext = ( (SessionImplementor) s ).getPersistenceContext();
final ActionQueue actionQueue = ( (SessionImpl) s ).getActionQueue();
assertEquals( 1, persistenceContext.getCollectionEntries().size() );
assertEquals( 1, persistenceContext.getCollectionEntriesSize() );
assertEquals( 1, persistenceContext.getCollectionsByKey().size() );
assertTrue( persistenceContext.getCollectionEntries().containsKey( publisher.getAuthors() ) );
assertTrue( persistenceContext.getCollectionsByKey().values().contains( publisher.getAuthors() ) );
@ -73,7 +73,7 @@ public void testAutoflushIsRequired() {
"autoflush collection update",
s.createQuery( "select a from Publisher p join p.authors a" ).list().size() == 1
);
assertEquals( 2, persistenceContext.getCollectionEntries().size() );
assertEquals( 2, persistenceContext.getCollectionEntriesSize() );
assertEquals( 2, persistenceContext.getCollectionsByKey().size() );
assertTrue( persistenceContext.getCollectionEntries().containsKey( publisher.getAuthors() ) );
assertTrue( persistenceContext.getCollectionEntries().containsKey( author1.getBooks() ) );
@ -88,7 +88,7 @@ public void testAutoflushIsRequired() {
assertTrue( "autoflush collection update",
s.createQuery( "select a from Publisher p join p.authors a" ).list().size() == 0
);
assertEquals( 1, persistenceContext.getCollectionEntries().size() );
assertEquals( 1, persistenceContext.getCollectionEntriesSize() );
assertEquals( 1, persistenceContext.getCollectionsByKey().size() );
assertTrue( persistenceContext.getCollectionEntries().containsKey( publisher.getAuthors() ) );
assertTrue( persistenceContext.getCollectionsByKey().values().contains( publisher.getAuthors() ) );
@ -102,7 +102,7 @@ public void testAutoflushIsRequired() {
publisher.getAuthors().add( author2 );
List results = s.createQuery( "select a from Publisher p join p.authors a" ).list();
assertEquals( 1, results.size() );
assertEquals( 2, persistenceContext.getCollectionEntries().size() );
assertEquals( 2, persistenceContext.getCollectionEntriesSize() );
assertEquals( 2, persistenceContext.getCollectionsByKey().size() );
assertTrue( persistenceContext.getCollectionEntries().containsKey( publisher.getAuthors() ) );
assertTrue( persistenceContext.getCollectionEntries().containsKey( author2.getBooks() ) );
@ -139,7 +139,7 @@ public void testAutoflushIsNotRequiredWithUnrelatedCollectionChange() {
final PersistenceContext persistenceContext = ( (SessionImplementor) s ).getPersistenceContext();
final ActionQueue actionQueue = ( (SessionImpl) s ).getActionQueue();
assertEquals( 1, persistenceContext.getCollectionEntries().size() );
assertEquals( 1, persistenceContext.getCollectionEntriesSize() );
assertEquals( 1, persistenceContext.getCollectionsByKey().size() );
assertTrue( persistenceContext.getCollectionEntries().containsKey( publisher.getAuthors() ) );
assertTrue( persistenceContext.getCollectionsByKey().values().contains( publisher.getAuthors() ) );
@ -149,7 +149,7 @@ public void testAutoflushIsNotRequiredWithUnrelatedCollectionChange() {
author1.setPublisher( publisher );
publisher.getAuthors().add( author1 );
assertTrue( s.createQuery( "from UnrelatedEntity" ).list().size() == 1 );
assertEquals( 2, persistenceContext.getCollectionEntries().size() );
assertEquals( 2, persistenceContext.getCollectionEntriesSize() );
assertEquals( 1, persistenceContext.getCollectionsByKey().size() );
assertTrue( persistenceContext.getCollectionEntries().containsKey( publisher.getAuthors() ) );
assertTrue( persistenceContext.getCollectionEntries().containsKey( author1.getBooks() ) );
@ -161,7 +161,7 @@ public void testAutoflushIsNotRequiredWithUnrelatedCollectionChange() {
publisher.getAuthors().clear();
assertEquals( 0, actionQueue.numberOfCollectionRemovals() );
assertTrue( s.createQuery( "from UnrelatedEntity" ).list().size() == 1 );
assertEquals( 2, persistenceContext.getCollectionEntries().size() );
assertEquals( 2, persistenceContext.getCollectionEntriesSize() );
assertEquals( 1, persistenceContext.getCollectionsByKey().size() );
assertTrue( persistenceContext.getCollectionEntries().containsKey( publisher.getAuthors() ) );
assertTrue( persistenceContext.getCollectionEntries().containsKey( author1.getBooks() ) );
@ -176,7 +176,7 @@ public void testAutoflushIsNotRequiredWithUnrelatedCollectionChange() {
publisher.getAuthors().add( author2 );
List results = s.createQuery( "from UnrelatedEntity" ).list();
assertEquals( 1, results.size() );
assertEquals( 4, persistenceContext.getCollectionEntries().size() );
assertEquals( 4, persistenceContext.getCollectionEntriesSize() );
assertEquals( 1, persistenceContext.getCollectionsByKey().size() );
assertTrue( persistenceContext.getCollectionEntries().containsKey( publisher.getAuthors() ) );
assertTrue( persistenceContext.getCollectionEntries().containsKey( author2.getBooks() ) );
@ -186,7 +186,7 @@ public void testAutoflushIsNotRequiredWithUnrelatedCollectionChange() {
assertEquals( 0, actionQueue.numberOfCollectionRemovals() );
s.flush();
assertEquals( 2, persistenceContext.getCollectionEntries().size() );
assertEquals( 2, persistenceContext.getCollectionEntriesSize() );
assertEquals( 2, persistenceContext.getCollectionsByKey().size() );
assertTrue( persistenceContext.getCollectionEntries().containsKey( publisher.getAuthors() ) );
assertTrue( persistenceContext.getCollectionEntries().containsKey( author2.getBooks() ) );