You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by aw...@apache.org on 2009/02/11 21:14:25 UTC

svn commit: r743477 - /openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/BrokerImpl.java

Author: awhite
Date: Wed Feb 11 20:14:24 2009
New Revision: 743477

URL: http://svn.apache.org/viewvc?rev=743477&view=rev
Log:
OPENJPA-913


Modified:
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/BrokerImpl.java

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/BrokerImpl.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/BrokerImpl.java?rev=743477&r1=743476&r2=743477&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/BrokerImpl.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/BrokerImpl.java Wed Feb 11 20:14:24 2009
@@ -778,8 +778,15 @@
         int eventType) {
         if (_lifeEventManager == null)
             return false;
-        handleCallbackExceptions(_lifeEventManager.fireEvent(src, related, 
-            meta, eventType), _lifeCallbackMode);
+
+        lock();
+        Exception[] exs;
+        try {
+            exs = _lifeEventManager.fireEvent(src, related, meta, eventType);
+        } finally {
+            unlock();
+        } 
+        handleCallbackExceptions(exs, _lifeCallbackMode);
         return true;
     }