[jboss-svn-commits] JBL Code SVN: r26058 - labs/jbosstm/trunk/ArjunaJTA/jta/classes/com/arjuna/ats/internal/jta/resources/arjunacore.

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

Author: mark.little at jboss.com
Date: 2009-04-16 04:29:06 -0400 (Thu, 16 Apr 2009)
New Revision: 26058

Modified:
   labs/jbosstm/trunk/ArjunaJTA/jta/classes/com/arjuna/ats/internal/jta/resources/arjunacore/XAResourceRecord.java
Log:
comment clarification.

Modified: labs/jbosstm/trunk/ArjunaJTA/jta/classes/com/arjuna/ats/internal/jta/resources/arjunacore/XAResourceRecord.java
===================================================================
--- labs/jbosstm/trunk/ArjunaJTA/jta/classes/com/arjuna/ats/internal/jta/resources/arjunacore/XAResourceRecord.java	2009-04-16 08:10:38 UTC (rev 26057)
+++ labs/jbosstm/trunk/ArjunaJTA/jta/classes/com/arjuna/ats/internal/jta/resources/arjunacore/XAResourceRecord.java	2009-04-16 08:29:06 UTC (rev 26058)
@@ -621,6 +621,11 @@
 						        return TwoPhaseOutcome.HEURISTIC_HAZARD;  // something terminated the transaction!
 						case XAException.XAER_PROTO:
 						case XAException.XA_RETRY:
+	                                            /*
+                                                     * Could do timeout retry here, but that could cause other resources in the list to go down the
+                                                     * heuristic path (some are far too keen to do this). Fail and let recovery retry. Meanwhile
+                                                     * the coordinator will continue to commit the other resources immediately.
+                                                     */
 							return TwoPhaseOutcome.FINISH_ERROR;
 						case XAException.XAER_INVAL:
 						case XAException.XAER_RMFAIL: // resource manager