Ken Giusti commented on PROTON-961:

Let me retract that - this isn't the correct fix.

I tried hacking a copy of messenger to set the max-frame on its connections to 
128 bytes.  A few of the unit tests start failing with that setting, so I 
applied Gordon's fix above.  His fix prevents a few of the failures, but 
proton_tests.messenger.MessengerTest.testSendReceive1M fails.

Turns out even with that patch Messenger will hang for messages >= 1meg.  This 
is caused by the incoming-window closing on the receiver.  Since the transfers 
are partial, messenger never calls pn_link_advance or pn_link_recv, which 
replenishes the incoming window.

> messenger doesn't handle multi-frame transfers
> ----------------------------------------------
>                 Key: PROTON-961
>                 URL: https://issues.apache.org/jira/browse/PROTON-961
>             Project: Qpid Proton
>          Issue Type: Bug
>          Components: proton-c
>    Affects Versions: 0.10, 0.11
>            Reporter: Gordon Sim
> See QPID-6651 for a reproducer.

This message was sent by Atlassian JIRA

Reply via email to