geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From djen...@apache.org
Subject svn commit: rev 54829 - geronimo/trunk/modules/transaction/src/java/org/apache/geronimo/transaction/manager
Date Fri, 15 Oct 2004 07:06:10 GMT
Author: djencks
Date: Fri Oct 15 00:06:08 2004
New Revision: 54829

Modified:
   geronimo/trunk/modules/transaction/src/java/org/apache/geronimo/transaction/manager/TransactionImpl.java
Log:
If XAResource.prepare throws an XAException, we can assume it is rolled back

Modified: geronimo/trunk/modules/transaction/src/java/org/apache/geronimo/transaction/manager/TransactionImpl.java
==============================================================================
--- geronimo/trunk/modules/transaction/src/java/org/apache/geronimo/transaction/manager/TransactionImpl.java
(original)
+++ geronimo/trunk/modules/transaction/src/java/org/apache/geronimo/transaction/manager/TransactionImpl.java
Fri Oct 15 00:06:08 2004
@@ -351,6 +351,10 @@
             } catch (XAException e) {
                 synchronized (this) {
                     status = Status.STATUS_MARKED_ROLLBACK;
+                    //TODO document why this is true from the spec.
+                    //XAException during prepare means we can assume resource is rolled back.
+                    rms.remove();
+                    break;
                 }
             }
         }

Mime
View raw message