[jboss-cvs] JBoss Messaging SVN: r3516 - trunk/src/main/org/jboss/jms/server/endpoint/advised.

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

Author: jmesnil
Date: 2007-12-17 12:17:36 -0500 (Mon, 17 Dec 2007)
New Revision: 3516

Modified:
   trunk/src/main/org/jboss/jms/server/endpoint/advised/SessionAdvised.java
Log:
* following modification to ClientSessionDelegate.send() method in r3514, a NullPacket is sent back as a response to a SendMessage only when sequence == -1 (msg is reliable or strict TCL is enabled)

Modified: trunk/src/main/org/jboss/jms/server/endpoint/advised/SessionAdvised.java
===================================================================
--- trunk/src/main/org/jboss/jms/server/endpoint/advised/SessionAdvised.java	2007-12-17 15:58:08 UTC (rev 3515)
+++ trunk/src/main/org/jboss/jms/server/endpoint/advised/SessionAdvised.java	2007-12-17 17:17:36 UTC (rev 3516)
@@ -239,10 +239,16 @@
             if (type == MSG_SENDMESSAGE)
             {
                SendMessage message = (SendMessage) packet;
-               send(message.getMessage(), message.checkForDuplicates(), message
-                     .getSequence());
+           
+               long sequence = message.getSequence(); 
+               send(message.getMessage(), message.checkForDuplicates(), sequence);
 
-               response = new NullPacket();
+               // a response is required only if seq == -1 -> reliable message or strict TCK
+               if (sequence == -1)
+               {               
+                  response = new NullPacket();
+               }
+               
             } else if (type == REQ_CREATECONSUMER)
             {
                CreateConsumerRequest request = (CreateConsumerRequest) packet;