You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@aries.apache.org by cs...@apache.org on 2017/02/21 10:09:50 UTC
svn commit: r1783861 -
/aries/trunk/jpa/jpa-blueprint/src/main/java/org/apache/aries/jpa/blueprint/impl/JpaInterceptor.java
Author: cschneider
Date: Tue Feb 21 10:09:50 2017
New Revision: 1783861
URL: http://svn.apache.org/viewvc?rev=1783861&view=rev
Log:
[ARIES-1689] Improve performance
Modified:
aries/trunk/jpa/jpa-blueprint/src/main/java/org/apache/aries/jpa/blueprint/impl/JpaInterceptor.java
Modified: aries/trunk/jpa/jpa-blueprint/src/main/java/org/apache/aries/jpa/blueprint/impl/JpaInterceptor.java
URL: http://svn.apache.org/viewvc/aries/trunk/jpa/jpa-blueprint/src/main/java/org/apache/aries/jpa/blueprint/impl/JpaInterceptor.java?rev=1783861&r1=1783860&r2=1783861&view=diff
==============================================================================
--- aries/trunk/jpa/jpa-blueprint/src/main/java/org/apache/aries/jpa/blueprint/impl/JpaInterceptor.java (original)
+++ aries/trunk/jpa/jpa-blueprint/src/main/java/org/apache/aries/jpa/blueprint/impl/JpaInterceptor.java Tue Feb 21 10:09:50 2017
@@ -53,10 +53,8 @@ public class JpaInterceptor implements I
@Override
public Object preCall(ComponentMetadata cm, Method m, Object... parameters) throws Throwable {
- synchronized (this) {
- if (coordinator == null) {
- initServices();
- }
+ if (coordinator == null) {
+ initServices();
}
try {
LOG.debug("PreCall for bean {}, method {}", cm.getId(), m.getName());
@@ -72,9 +70,11 @@ public class JpaInterceptor implements I
}
}
- private void initServices() {
- coordinator = (Coordinator)container.getComponentInstance(coordinatorId);
- em = (EntityManager)container.getComponentInstance(emId);
+ private synchronized void initServices() {
+ if (coordinator == null) {
+ coordinator = (Coordinator)container.getComponentInstance(coordinatorId);
+ em = (EntityManager)container.getComponentInstance(emId);
+ }
}
@Override