[jboss-svn-commits] JBL Code SVN: r35329 - labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi.

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

Author: adinn
Date: 2010-09-28 04:36:56 -0400 (Tue, 28 Sep 2010)
New Revision: 35329

Modified:
   labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi/TaxiManager.java
   labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi/TaxiParticipantAT.java
   labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi/TaxiParticipantBA.java
Log:
updated some of the javadoc -- fixes for JBTM-790

Modified: labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi/TaxiManager.java
===================================================================
--- labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi/TaxiManager.java	2010-09-28 08:25:26 UTC (rev 35328)
+++ labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi/TaxiManager.java	2010-09-28 08:36:56 UTC (rev 35329)
@@ -36,14 +36,15 @@
  * The application logic for the Taxi Service
  * <p/>
  * Manages taxi reservations, providing prepare, commit and rollback calls for
- * modifying taxi bookings in memory. Taxis are an unlimited resource and it does
+ * modifying bookings <em>in memory only</em>. Taxis are an unlimited resource and it does
  * not really matter if a taxi does not turn up (there's always another one round the
  * corner) nor does it matter of the clients don't appear (someone else will take
- * the ride). so this manager does not maintain any persistent state and any bookings
- * made are not resilient to crashes.
+ * the ride). so this manager does not maintain any persistent state and the bookings
+ * it makes are not resilient to crashes.
  *
  * @author Jonathan Halliday (jonathan.halliday at arjuna.com)
- * @version $Revision: 1.3 $
+ * @author Andrew Dinn (adinn at redhat.com)
+ * @version $Revision: 1.3$
  */
 public class TaxiManager implements Serializable
 {
@@ -68,7 +69,6 @@
 
     /**
      * Book a taxi.
-     *
      * @param txID The transaction identifier
      */
     public synchronized void bookTaxi(Object txID)
@@ -88,6 +88,7 @@
 
     /**
      * check whether we have already seen a web service request in a given transaction
+     * @param txID The transaction identifier
      */
 
     public synchronized boolean knowsAbout(Object txID)
@@ -111,7 +112,6 @@
         Integer request = (Integer) transactions.get(txID);
         if (request == null)
         {
-            transactions.remove(txID);
             return false;
         }
         else
@@ -130,7 +130,7 @@
     /**
      * commit local state changes for the supplied transaction
      *
-     * @param txID
+     * @param txID The transaction identifier
      */
     public synchronized void commit(Object txID)
     {
@@ -141,7 +141,7 @@
     /**
      * roll back local state changes for the supplied transaction
      *
-     * @param txID
+     * @param txID The transaction identifier
      */
     public synchronized void rollback(Object txID)
     {

Modified: labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi/TaxiParticipantAT.java
===================================================================
--- labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi/TaxiParticipantAT.java	2010-09-28 08:25:26 UTC (rev 35328)
+++ labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi/TaxiParticipantAT.java	2010-09-28 08:36:56 UTC (rev 35329)
@@ -154,12 +154,12 @@
 
     public void unknown() throws SystemException
     {
-        // used for calbacks during crash recovery. This impl is not recoverable
+        // used for callbacks during crash recovery. This impl is not recoverable
     }
 
     public void error() throws SystemException
     {
-        // used for calbacks during crash recovery. This impl is not recoverable
+        // used for callbacks during crash recovery. This impl is not recoverable
     }
 
     /**

Modified: labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi/TaxiParticipantBA.java
===================================================================
--- labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi/TaxiParticipantBA.java	2010-09-28 08:25:26 UTC (rev 35328)
+++ labs/jbosstm/trunk/XTS/demo/src/com/jboss/jbosstm/xts/demo/services/taxi/TaxiParticipantBA.java	2010-09-28 08:36:56 UTC (rev 35329)
@@ -29,7 +29,11 @@
 
 package com.jboss.jbosstm.xts.demo.services.taxi;
 
-import com.arjuna.wst.*;
+import com.arjuna.wst.BusinessAgreementWithCoordinatorCompletionParticipant;
+import com.arjuna.wst.FaultedException;
+import com.arjuna.wst.SystemException;
+import com.arjuna.wst.WrongStateException;
+
 import com.arjuna.wst11.BAParticipantManager;
 import com.arjuna.wst11.ConfirmCompletedParticipant;
 
@@ -191,7 +195,7 @@
 
     public void unknown() throws SystemException
     {
-        // used for calbacks during crash recovery. This impl is not recoverable
+        // used for callbacks during crash recovery. This impl is not recoverable
     }
 
     public void error() throws SystemException
@@ -246,7 +250,7 @@
      * @param txID the participant's transaction id
      * @param participant
      */
-    public static synchronized void recordParticipant(String txID, TaxiParticipantBA participant, com.arjuna.wst11.BAParticipantManager manager)
+    public static synchronized void recordParticipant(String txID, TaxiParticipantBA participant, BAParticipantManager manager)
     {
         participants.put(txID, participant);
         managers.put(txID, manager);
@@ -277,7 +281,7 @@
      * @param txID the participant's transaction id
      * @return the participant's manager
      */
-    public static synchronized com.arjuna.wst11.BAParticipantManager getManager(String txID)
+    public static synchronized BAParticipantManager getManager(String txID)
     {
         return managers.get(txID);
     }
@@ -307,5 +311,5 @@
     /**
      * table of currently active participant managers
      */
-    private static HashMap<String, com.arjuna.wst11.BAParticipantManager> managers = new HashMap<String, com.arjuna.wst11.BAParticipantManager>();
+    private static HashMap<String, BAParticipantManager> managers = new HashMap<String, BAParticipantManager>();
 }