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

HHH-13179 Fixed code formatting issue

This commit is contained in:
Lars Karlström 2020-02-19 14:45:24 +01:00 committed by Andrea Boriero
parent 48b53cbb80
commit d28f513849
2 changed files with 34 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
hibernate-core/src/main/java/org/hibernate/boot/model/source/internal/hbm/ModelBinder.java
View File

@ -523,7 +523,7 @@ public class ModelBinder {
PersistentClass superEntityDescriptor) {
for ( IdentifiableTypeSource subType : entitySource.getSubTypes() ) {
final SingleTableSubclass subEntityDescriptor = new SingleTableSubclass( superEntityDescriptor, metadataBuildingContext );
subEntityDescriptor.setCached(superEntityDescriptor.isCached());
subEntityDescriptor.setCached( superEntityDescriptor.isCached() );
bindDiscriminatorSubclassEntity( (SubclassEntitySourceImpl) subType, subEntityDescriptor );
superEntityDescriptor.addSubclass( subEntityDescriptor );
entitySource.getLocalMetadataBuildingContext().getMetadataCollector().addEntityBinding( subEntityDescriptor );
@ -582,7 +582,7 @@ public class ModelBinder {
PersistentClass superEntityDescriptor) {
for ( IdentifiableTypeSource subType : entitySource.getSubTypes() ) {
final JoinedSubclass subEntityDescriptor = new JoinedSubclass( superEntityDescriptor, metadataBuildingContext );
subEntityDescriptor.setCached(superEntityDescriptor.isCached());
subEntityDescriptor.setCached( superEntityDescriptor.isCached() );
bindJoinedSubclassEntity( (JoinedSubclassEntitySourceImpl) subType, subEntityDescriptor );
superEntityDescriptor.addSubclass( subEntityDescriptor );
entitySource.getLocalMetadataBuildingContext().getMetadataCollector().addEntityBinding( subEntityDescriptor );
@ -659,7 +659,7 @@ public class ModelBinder {
PersistentClass superEntityDescriptor) {
for ( IdentifiableTypeSource subType : entitySource.getSubTypes() ) {
final UnionSubclass subEntityDescriptor = new UnionSubclass( superEntityDescriptor, metadataBuildingContext );
subEntityDescriptor.setCached(superEntityDescriptor.isCached());
subEntityDescriptor.setCached( superEntityDescriptor.isCached() );
bindUnionSubclassEntity( (SubclassEntitySourceImpl) subType, subEntityDescriptor );
superEntityDescriptor.addSubclass( subEntityDescriptor );
entitySource.getLocalMetadataBuildingContext().getMetadataCollector().addEntityBinding( subEntityDescriptor );

57
hibernate-core/src/test/java/org/hibernate/test/cache/hhh13179/HHH13179Test.java vendored
View File

@ -20,6 +20,7 @@ import org.hibernate.Transaction;
import org.hibernate.cfg.AvailableSettings;
import org.hibernate.cfg.Configuration;
import org.hibernate.stat.CacheRegionStatistics;
import org.hibernate.testing.TestForIssue;
import org.hibernate.testing.junit4.BaseCoreFunctionalTestCase;
import org.junit.Assert;
@ -28,7 +29,7 @@ import org.junit.Test;
/**
* Check that second level caching works for hbm mapped joined subclass inheritance structures
*/
@TestForIssue( jiraKey = "HHH-13179")
@TestForIssue(jiraKey = "HHH-13179")
public class HHH13179Test extends BaseCoreFunctionalTestCase {
// Add your entities here.
@ -56,6 +57,7 @@ public class HHH13179Test extends BaseCoreFunctionalTestCase {
"org/hibernate/test/cache/hhh13179/DiscriminatorSubclassPerson.hbm.xml"
};
}
// If those mappings reside somewhere other than resources/org/hibernate/test, change this.
@Override
protected String getBaseForMappings() {
@ -79,14 +81,15 @@ public class HHH13179Test extends BaseCoreFunctionalTestCase {
Transaction tx = s.beginTransaction();
String regionName = "org.hibernate.test.cache.hhh13179.JoinedSubclassPerson";
// sanity check
CacheRegionStatistics cacheRegionStatistics = s.getSessionFactory().getStatistics().getCacheRegionStatistics(regionName);
Assert.assertEquals("Cache put should be 0", 0, cacheRegionStatistics.getPutCount());
CacheRegionStatistics cacheRegionStatistics = s.getSessionFactory().getStatistics().getCacheRegionStatistics(
regionName );
Assert.assertEquals( "Cache put should be 0", 0, cacheRegionStatistics.getPutCount() );
JoinedSubclassPerson person1 = new JoinedSubclassUIPerson();
person1.setOid(1L);
s.save(person1);
person1.setOid( 1L );
s.save( person1 );
tx.commit();
@ -95,11 +98,11 @@ public class HHH13179Test extends BaseCoreFunctionalTestCase {
s = openSession();
tx = s.beginTransaction();
JoinedSubclassPerson person2 = s.get(JoinedSubclassPerson.class, 1L);
JoinedSubclassPerson person2 = s.get( JoinedSubclassPerson.class, 1L );
cacheRegionStatistics = s.getSessionFactory().getStatistics().getCacheRegionStatistics(regionName);
Assert.assertEquals("Cache hit should be 1", 1, cacheRegionStatistics.getHitCount());
Assert.assertEquals("Cache put should be 1", 1, cacheRegionStatistics.getPutCount());
cacheRegionStatistics = s.getSessionFactory().getStatistics().getCacheRegionStatistics( regionName );
Assert.assertEquals( "Cache hit should be 1", 1, cacheRegionStatistics.getHitCount() );
Assert.assertEquals( "Cache put should be 1", 1, cacheRegionStatistics.getPutCount() );
tx.commit();
s.close();
@ -114,12 +117,13 @@ public class HHH13179Test extends BaseCoreFunctionalTestCase {
String regionName = "org.hibernate.test.cache.hhh13179.UnionSubclassPerson";
// sanity check
CacheRegionStatistics cacheRegionStatistics = s.getSessionFactory().getStatistics().getCacheRegionStatistics(regionName);
Assert.assertEquals("Cache put should be 0", 0, cacheRegionStatistics.getPutCount());
CacheRegionStatistics cacheRegionStatistics = s.getSessionFactory().getStatistics().getCacheRegionStatistics(
regionName );
Assert.assertEquals( "Cache put should be 0", 0, cacheRegionStatistics.getPutCount() );
UnionSubclassPerson person1 = new UnionSubclassUIPerson();
person1.setOid(1L);
s.save(person1);
person1.setOid( 1L );
s.save( person1 );
tx.commit();
@ -128,11 +132,11 @@ public class HHH13179Test extends BaseCoreFunctionalTestCase {
s = openSession();
tx = s.beginTransaction();
UnionSubclassPerson person2 = s.get(UnionSubclassPerson.class, 1L);
UnionSubclassPerson person2 = s.get( UnionSubclassPerson.class, 1L );
cacheRegionStatistics = s.getSessionFactory().getStatistics().getCacheRegionStatistics(regionName);
Assert.assertEquals("Cache hit should be 1", 1, cacheRegionStatistics.getHitCount());
Assert.assertEquals("Cache put should be 1", 1, cacheRegionStatistics.getPutCount());
cacheRegionStatistics = s.getSessionFactory().getStatistics().getCacheRegionStatistics( regionName );
Assert.assertEquals( "Cache hit should be 1", 1, cacheRegionStatistics.getHitCount() );
Assert.assertEquals( "Cache put should be 1", 1, cacheRegionStatistics.getPutCount() );
tx.commit();
s.close();
@ -147,12 +151,13 @@ public class HHH13179Test extends BaseCoreFunctionalTestCase {
String regionName = "org.hibernate.test.cache.hhh13179.DiscriminatorSubclassPerson";
// sanity check
CacheRegionStatistics cacheRegionStatistics = s.getSessionFactory().getStatistics().getCacheRegionStatistics(regionName);
Assert.assertEquals("Cache put should be 0", 0, cacheRegionStatistics.getPutCount());
CacheRegionStatistics cacheRegionStatistics = s.getSessionFactory().getStatistics().getCacheRegionStatistics(
regionName );
Assert.assertEquals( "Cache put should be 0", 0, cacheRegionStatistics.getPutCount() );
DiscriminatorSubclassPerson person1 = new DiscriminatorSubclassUIPerson();
person1.setOid(1L);
s.save(person1);
person1.setOid( 1L );
s.save( person1 );
tx.commit();
@ -161,11 +166,11 @@ public class HHH13179Test extends BaseCoreFunctionalTestCase {
s = openSession();
tx = s.beginTransaction();
DiscriminatorSubclassPerson person2 = s.get(DiscriminatorSubclassPerson.class, 1L);
DiscriminatorSubclassPerson person2 = s.get( DiscriminatorSubclassPerson.class, 1L );
cacheRegionStatistics = s.getSessionFactory().getStatistics().getCacheRegionStatistics(regionName);
Assert.assertEquals("Cache hit should be 1", 1, cacheRegionStatistics.getHitCount());
Assert.assertEquals("Cache put should be 1", 1, cacheRegionStatistics.getPutCount());
cacheRegionStatistics = s.getSessionFactory().getStatistics().getCacheRegionStatistics( regionName );
Assert.assertEquals( "Cache hit should be 1", 1, cacheRegionStatistics.getHitCount() );
Assert.assertEquals( "Cache put should be 1", 1, cacheRegionStatistics.getPutCount() );
tx.commit();
s.close();