[jboss-cvs] JBossAS SVN: r110048 - branches/JBPAPP_4_3_0_GA_CP08_JBPAPP-5432/transaction/src/main/org/jboss/tm.

[jboss-cvs-commits at lists.jboss.org]

Author: jhowell at redhat.com
Date: 2010-12-20 12:29:54 -0500 (Mon, 20 Dec 2010)
New Revision: 110048

Modified:
   branches/JBPAPP_4_3_0_GA_CP08_JBPAPP-5432/transaction/src/main/org/jboss/tm/TransactionImpl.java
Log:
One phase commit resources were not being rolled back when they should with the new fix.  I added the rollbackresources call to the one phase commit process.

Modified: branches/JBPAPP_4_3_0_GA_CP08_JBPAPP-5432/transaction/src/main/org/jboss/tm/TransactionImpl.java
===================================================================
--- branches/JBPAPP_4_3_0_GA_CP08_JBPAPP-5432/transaction/src/main/org/jboss/tm/TransactionImpl.java	2010-12-20 17:26:36 UTC (rev 110047)
+++ branches/JBPAPP_4_3_0_GA_CP08_JBPAPP-5432/transaction/src/main/org/jboss/tm/TransactionImpl.java	2010-12-20 17:29:54 UTC (rev 110048)
@@ -368,9 +368,7 @@
          if (status != Status.STATUS_COMMITTED)
          {
             Throwable causedByThrowable = cause;
-            if (getCommitStrategy() == 2) {
-               rollbackResources();
-            }
+            rollbackResources();
             completeTransaction();
             checkHeuristics();
             // throw jboss rollback exception with the saved off cause