[jbpm-commits] JBoss JBPM SVN: r2910 - projects/gwt-console/trunk/server/src/main/java/org/jboss/bpm/console/server/dao.

[do-not-reply at jboss.org]

Author: heiko.braun at jboss.com
Date: 2008-11-13 09:08:35 -0500 (Thu, 13 Nov 2008)
New Revision: 2910

Modified:
   projects/gwt-console/trunk/server/src/main/java/org/jboss/bpm/console/server/dao/DAOProxy.java
Log:
Catch runtime exceptions in DAOProxy. Force setRollbackOnly()

Modified: projects/gwt-console/trunk/server/src/main/java/org/jboss/bpm/console/server/dao/DAOProxy.java
===================================================================
--- projects/gwt-console/trunk/server/src/main/java/org/jboss/bpm/console/server/dao/DAOProxy.java	2008-11-13 14:07:45 UTC (rev 2909)
+++ projects/gwt-console/trunk/server/src/main/java/org/jboss/bpm/console/server/dao/DAOProxy.java	2008-11-13 14:08:35 UTC (rev 2910)
@@ -22,6 +22,8 @@
 package org.jboss.bpm.console.server.dao;
 
 import org.jboss.bpm.console.server.util.ServiceLocator;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 
 import javax.transaction.SystemException;
 import javax.transaction.UserTransaction;
@@ -34,6 +36,8 @@
  */
 public class DAOProxy implements java.lang.reflect.InvocationHandler
 {
+   private static final Log log = LogFactory.getLog(DAOProxy.class);
+
    private Object obj;
 
    public static Object newInstance(Object obj) {
@@ -65,7 +69,7 @@
          tx.commit();
 
       }
-      catch (Exception e)
+      catch (Throwable e)
       {
          if(tx!=null)
          {
@@ -73,7 +77,9 @@
             {
                tx.rollback();
             }
-            catch (SystemException e1) {}
+            catch (SystemException e1) {
+               log.error("Failed to rollback on  " + obj + " -> " + m.getName());  
+            }
          }
 
          throw new RuntimeException("Unexpected invocation exception: " + e.getMessage(), e);