You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by rg...@apache.org on 2012/11/13 16:01:39 UTC

svn commit: r1408771 - /qpid/proton/trunk/proton-j/src/main/java/org/apache/qpid/proton/engine/impl/TransportImpl.java

Author: rgodfrey
Date: Tue Nov 13 15:01:38 2012
New Revision: 1408771

URL: http://svn.apache.org/viewvc?rev=1408771&view=rev
Log:
PROTON-130 : Sending pre-settled message results in the TransportSession._outgoingWindowSize going to 0, which stops further transfers.

Modified:
    qpid/proton/trunk/proton-j/src/main/java/org/apache/qpid/proton/engine/impl/TransportImpl.java

Modified: qpid/proton/trunk/proton-j/src/main/java/org/apache/qpid/proton/engine/impl/TransportImpl.java
URL: http://svn.apache.org/viewvc/qpid/proton/trunk/proton-j/src/main/java/org/apache/qpid/proton/engine/impl/TransportImpl.java?rev=1408771&r1=1408770&r2=1408771&view=diff
==============================================================================
--- qpid/proton/trunk/proton-j/src/main/java/org/apache/qpid/proton/engine/impl/TransportImpl.java (original)
+++ qpid/proton/trunk/proton-j/src/main/java/org/apache/qpid/proton/engine/impl/TransportImpl.java Tue Nov 13 15:01:38 2012
@@ -391,7 +391,6 @@ public class TransportImpl extends Endpo
                     UnsignedInteger deliveryId = transportLink.getDeliveryCount();
                     TransportDelivery transportDelivery = new TransportDelivery(deliveryId, delivery, transportLink);
                     delivery.setTransportDelivery(transportDelivery);
-                    sender.getSession().getTransportSession().addUnsettledOutgoing(deliveryId, delivery);
 
                     final Transfer transfer = new Transfer();
                     transfer.setDeliveryId(deliveryId);
@@ -402,6 +401,11 @@ public class TransportImpl extends Endpo
                     {
                         transfer.setSettled(Boolean.TRUE);
                     }
+                    else
+                    {
+                        sender.getSession().getTransportSession().addUnsettledOutgoing(deliveryId, delivery);
+                    }
+
                     if(delivery.getLink().current() == delivery)
                     {
                         transfer.setMore(true);



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org