-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39561/
-----------------------------------------------------------

(Updated Oct. 23, 2015, 9:46 a.m.)


Review request for qpid, Alan Conway and Kenneth Giusti.


Changes
-------

Small addition to previous update, ensuring that the disharge message is marked 
as pending.


Bugs: QPID-6790
    https://issues.apache.org/jira/browse/QPID-6790


Repository: qpid


Description
-------

This is an alterntaive fix that simply shortcircuits the asynchornous 
completion for deliveries whose link has already been freed. It does this by 
tracking deliveries for which asynchronous acceptance is pending, and on 
deleting links removes the records of any deliveries affected. On handling an 
asynchronous accept, the epnding set is consulted first and the accept only 
proceeds if it is found.


Diffs (updated)
-----

  trunk/qpid/cpp/src/qpid/broker/amqp/Incoming.cpp 1710066 
  trunk/qpid/cpp/src/qpid/broker/amqp/Session.h 1710066 
  trunk/qpid/cpp/src/qpid/broker/amqp/Session.cpp 1710066 

Diff: https://reviews.apache.org/r/39561/diff/


Testing
-------

Passes make test.


Thanks,

Gordon Sim

Reply via email to