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

Author: heiko.braun(a)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);