Qpid
  1. Qpid
  2. QPID-3440

the reply-to Destination cache in AMQMessageDelegate_0_10 does not work

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.6, 0.7, 0.8, 0.9, 0.10, 0.11, 0.12, 0.13, 0.14
    • Fix Version/s: 0.15
    • Component/s: Java Client
    • Labels:
      None

      Description

      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.

        Issue Links

          Activity

          Robbie Gemmell created issue -
          Robbie Gemmell made changes -
          Field Original Value New Value
          Status Open [ 1 ] In Progress [ 3 ]
          Robbie Gemmell made changes -
          Attachment QPID-3440_wip.patch [ 12491164 ]
          Robbie Gemmell made changes -
          Link This issue is related to QPID-3016 [ QPID-3016 ]
          Robbie Gemmell made changes -
          Attachment QPID-3440_wip.patch [ 12491164 ]
          Robbie Gemmell made changes -
          Comment [ 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. ]
          Robbie Gemmell made changes -
          Fix Version/s 0.15 [ 12319043 ]
          Fix Version/s 0.13 [ 12316854 ]
          Affects Version/s 0.13 [ 12316854 ]
          Affects Version/s 0.14 [ 12316855 ]
          Robbie Gemmell made changes -
          Summary the Destination cache in AMQMessageDelegate_0_10 does not work the reply-to Destination cache in AMQMessageDelegate_0_10 does not work
          Robbie Gemmell made changes -
          Status In Progress [ 3 ] Ready To Review [ 10006 ]
          Robbie Gemmell made changes -
          Assignee Robbie Gemmell [ gemmellr ] Keith Wall [ k-wall ]
          Robbie Gemmell made changes -
          Status Ready To Review [ 10006 ] Resolved [ 5 ]
          Assignee Keith Wall [ k-wall ] Rob Godfrey [ rgodfrey ]
          Resolution Fixed [ 1 ]

            People

            • Assignee:
              Rob Godfrey
              Reporter:
              Robbie Gemmell
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development