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

HHH-15693 Introduce a fast-path access for ClassLoaderService being retrieved from ServiceRegistry

This commit is contained in:
Sanne Grinovero 2022-11-08 14:51:30 +00:00 committed by Sanne Grinovero
parent 1eeccd32eb
commit be2999d054
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
hibernate-core/src/main/java/org/hibernate/service/internal/AbstractServiceRegistryImpl.java
View File

@ -17,6 +17,7 @@ import java.util.concurrent.atomic.AtomicBoolean;
import java.util.function.Consumer; import java.util.function.Consumer;
import org.hibernate.boot.registry.BootstrapServiceRegistry; import org.hibernate.boot.registry.BootstrapServiceRegistry;
import org.hibernate.boot.registry.classloading.spi.ClassLoaderService;
import org.hibernate.cfg.Environment; import org.hibernate.cfg.Environment;
import org.hibernate.internal.CoreLogging; import org.hibernate.internal.CoreLogging;
import org.hibernate.internal.CoreMessageLogger; import org.hibernate.internal.CoreMessageLogger;
@ -184,6 +185,14 @@ public abstract class AbstractServiceRegistryImpl
@Override @Override
public <R extends Service> R getService(Class<R> serviceRole) { public <R extends Service> R getService(Class<R> serviceRole) {
//Fast-path for ClassLoaderService as it's extremely hot during bootstrap
//(and after bootstrap service loading performance is less interesting as it's
//ideally being cached by long term consumers)
if ( ClassLoaderService.class.equals( serviceRole ) ) {
if ( parent != null ) {
return parent.getService( serviceRole );
}
}
// TODO: should an exception be thrown if active == false??? // TODO: should an exception be thrown if active == false???
R service = serviceRole.cast( initializedServiceByRole.get( serviceRole ) ); R service = serviceRole.cast( initializedServiceByRole.get( serviceRole ) );
if ( service != null ) { if ( service != null ) {