You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by cu...@apache.org on 2010/09/20 21:28:47 UTC

svn commit: r999077 - in /openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence: EntityManagerFactoryImpl.java EntityManagerImpl.java PersistenceProviderImpl.java

Author: curtisr7
Date: Mon Sep 20 19:28:46 2010
New Revision: 999077

URL: http://svn.apache.org/viewvc?rev=999077&view=rev
Log:
OPENJPA-1807: Add trace to EM/EMF create/close path.

Modified:
    openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerFactoryImpl.java
    openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerImpl.java
    openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/PersistenceProviderImpl.java

Modified: openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerFactoryImpl.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerFactoryImpl.java?rev=999077&r1=999076&r2=999077&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerFactoryImpl.java (original)
+++ openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerFactoryImpl.java Mon Sep 20 19:28:46 2010
@@ -237,6 +237,9 @@ public class EntityManagerFactoryImpl
         for (Object key : props.keySet()) {
             em.setProperty(key.toString(), props.get(key));
         }
+        if (log != null && log.isTraceEnabled()) {
+            log.trace(this + " created EntityManager " + em + ".");
+        }
         return em;
     }
     
@@ -264,6 +267,10 @@ public class EntityManagerFactoryImpl
     }
 
     public void close() {
+        Log log = _factory.getConfiguration().getLog(OpenJPAConfiguration.LOG_RUNTIME);
+        if (log.isTraceEnabled()) {
+            log.trace(this + ".close() invoked.");
+        }
         _factory.close();
     }
 

Modified: openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerImpl.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerImpl.java?rev=999077&r1=999076&r2=999077&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerImpl.java (original)
+++ openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/EntityManagerImpl.java Mon Sep 20 19:28:46 2010
@@ -1275,6 +1275,10 @@ public class EntityManagerImpl
 
     public void close() {
         assertNotCloseInvoked();
+        Log log = _emf.getConfiguration().getLog(OpenJPAConfiguration.LOG_RUNTIME);
+        if (log.isTraceEnabled()) {
+            log.trace(this + ".close() invoked.");
+        }
         _broker.close();
         _plans.clear();
     }

Modified: openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/PersistenceProviderImpl.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/PersistenceProviderImpl.java?rev=999077&r1=999076&r2=999077&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/PersistenceProviderImpl.java (original)
+++ openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/PersistenceProviderImpl.java Mon Sep 20 19:28:46 2010
@@ -102,7 +102,11 @@ public class PersistenceProviderImpl
             // Create appropriate LifecycleEventManager
             loadValidator(factory);
             
-            return JPAFacadeHelper.toEntityManagerFactory(factory);
+            OpenJPAEntityManagerFactory emf = JPAFacadeHelper.toEntityManagerFactory(factory);
+            if (_log.isTraceEnabled()) {
+                _log.trace(this + " creating " + emf + " for PU " + name + ".");
+            }
+            return emf;
         } catch (Exception e) {
             if (_log != null) {
                 _log.error(_loc.get("create-emf-error", name), e);
@@ -196,7 +200,12 @@ public class PersistenceProviderImpl
             // Create appropriate LifecycleEventManager
             loadValidator(factory);
             
-            return JPAFacadeHelper.toEntityManagerFactory(factory);
+            OpenJPAEntityManagerFactory emf = JPAFacadeHelper.toEntityManagerFactory(factory);
+            if (_log.isTraceEnabled()) {
+                _log.trace(this + " creating container " + emf + " for PU " + pui.getPersistenceUnitName() + ".");
+            }
+            
+            return emf;
         } catch (Exception e) {
             throw PersistenceExceptions.toPersistenceException(e);
         }