The Destination cache in AMQMessageDelegate_0_10 does not work, because it indexes the cache by the underlying ReplyTo transport class, which has no equals() implementation.
|Field||Original Value||New Value|
|Status||Open [ 1 ]||In Progress [ 3 ]|
|Attachment||QPID-3440_wip.patch [ 12491164 ]|
[ Attaching an initial patch, with test to verify the issue and addition of equals() and hashCode() methods to the generated file causing the problem.
Not yet applied becuase doing so breaks PropertyValueTest. This is due to failing a Destination equality test, despite the fact the toString of the destinations is actually the same (which it wasnt before), but the equals check fails because of a change in underlying object in use, due to the horrible number of different objects and ways to instantiate them in our wonderfully dispirate Destination handling. ]
|Fix Version/s||0.15 [ 12319043 ]|
|Fix Version/s||0.13 [ 12316854 ]|
|Affects Version/s||0.13 [ 12316854 ]|
|Affects Version/s||0.14 [ 12316855 ]|
|Summary||the Destination cache in AMQMessageDelegate_0_10 does not work||the reply-to Destination cache in AMQMessageDelegate_0_10 does not work|
|Status||In Progress [ 3 ]||Ready To Review [ 10006 ]|
|Assignee||Robbie Gemmell [ gemmellr ]||Keith Wall [ k-wall ]|
|Status||Ready To Review [ 10006 ]||Resolved [ 5 ]|
|Assignee||Keith Wall [ k-wall ]||Rob Godfrey [ rgodfrey ]|
|Resolution||Fixed [ 1 ]|