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

HHH-13855 - Remove unnecessary delared variable JtaManager in HibernatePersistenceProviderAdaptor 

In the HibernatePersistenceProviderAdaptor, there is an unnecessary declaration of JtaManager. The sole purpose of having this variable is only to use in the overridden method named InjectJtaManager.

This was be improved by removing the JtaManager as there is no use of the variable, as of now at least.
This commit is contained in:
The Geeky Asian 2020-02-08 01:31:41 +04:00 committed by Sanne Grinovero
parent a7261ad053
commit 5501a40dd2
1 changed files with 1 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
hibernate-jipijapa/src/main/java/org/jboss/as/jpa/hibernate5/HibernatePersistenceProviderAdaptor.java
View File

@ -43,17 +43,11 @@ public class HibernatePersistenceProviderAdaptor implements PersistenceProviderA
@SuppressWarnings("WeakerAccess") @SuppressWarnings("WeakerAccess")
public static final String HIBERNATE_EXTENDED_BEANMANAGER = "org.hibernate.resource.beans.container.spi.ExtendedBeanManager"; public static final String HIBERNATE_EXTENDED_BEANMANAGER = "org.hibernate.resource.beans.container.spi.ExtendedBeanManager";
private volatile JtaManager jtaManager;
private volatile Platform platform; private volatile Platform platform;
private static final String NONE = SharedCacheMode.NONE.name(); private static final String NONE = SharedCacheMode.NONE.name();
@Override @Override
public void injectJtaManager(JtaManager jtaManager) { public void injectJtaManager(JtaManager jtaManager) { }
// todo : why? `this.jtaManager` is never used aside from setting here in this method
if ( this.jtaManager != jtaManager ) {
this.jtaManager = jtaManager;
}
}
@Override @Override
public void injectPlatform(Platform platform) { public void injectPlatform(Platform platform) {