You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ar...@apache.org on 2008/06/10 21:45:23 UTC

svn commit: r666296 - /incubator/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/BasicMessageConsumer_0_10.java

Author: arnaudsimon
Date: Tue Jun 10 12:45:23 2008
New Revision: 666296

URL: http://svn.apache.org/viewvc?rev=666296&view=rev
Log:
Qpid-1130: don't store unack message tags when the session is transacted

Modified:
    incubator/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/BasicMessageConsumer_0_10.java

Modified: incubator/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/BasicMessageConsumer_0_10.java
URL: http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/BasicMessageConsumer_0_10.java?rev=666296&r1=666295&r2=666296&view=diff
==============================================================================
--- incubator/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/BasicMessageConsumer_0_10.java (original)
+++ incubator/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/BasicMessageConsumer_0_10.java Tue Jun 10 12:45:23 2008
@@ -261,7 +261,10 @@
 
     @Override protected void preApplicationProcessing(AbstractJMSMessage jmsMsg) throws JMSException
     {
-        _session.addUnacknowledgedMessage(jmsMsg.getDeliveryTag());
+        if (!_session.getTransacted())
+        {
+            _session.addUnacknowledgedMessage(jmsMsg.getDeliveryTag());
+        }
         _session.setInRecovery(false);
     }