svn commit: r784599 - /webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/ServiceTaskManager.java

[ve...@apache.org]

Author: veithen
Date: Sun Jun 14 18:39:20 2009
New Revision: 784599

URL: http://svn.apache.org/viewvc?rev=784599&view=rev
Log:
Added review comments.

Modified:
    webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/ServiceTaskManager.java

Modified: webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/ServiceTaskManager.java
URL: http://svn.apache.org/viewvc/webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/ServiceTaskManager.java?rev=784599&r1=784598&r2=784599&view=diff
==============================================================================
--- webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/ServiceTaskManager.java (original)
+++ webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/ServiceTaskManager.java Sun Jun 14 18:39:20 2009
@@ -66,10 +66,12 @@
     /** The ConnectionFactory MUST refer to an XAConnectionFactory to use JTA */
     private String connFactoryJNDIName;
     /** The JNDI name of the Destination Queue or Topic */
+    // TODO: this overlaps with JMSEndpoint#jndiDestinationName; needs to be clarified
     private String destinationJNDIName;
     /** JNDI location for the JTA UserTransaction */
     private String userTransactionJNDIName = "java:comp/UserTransaction";
     /** The type of destination - P2P or PubSub (or JMS 1.1 API generic?) */
+    // TODO: this overlaps with JMSEndpoint#destinationType; needs to be clarified
     private int destinationType = JMSConstants.GENERIC;
     /** An optional message selector */
     private String messageSelector = null;