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

          Hide
          Robbie Gemmell added a comment -

          Closing out, Rob reviewed the commit while the new test was failing on his Sonar runs (due to a configuration issue with the Sonar build run)

          Show
          Robbie Gemmell added a comment - Closing out, Rob reviewed the commit while the new test was failing on his Sonar runs (due to a configuration issue with the Sonar build run)
          Hide
          Robbie Gemmell added a comment -

          Keith, could you review these changes please?

          Thanks,
          Robbie

          Show
          Robbie Gemmell added a comment - Keith, could you review these changes please? Thanks, Robbie
          Hide
          Robbie Gemmell added a comment -

          Added a test to verify the issue, and its resolution following addition of equals() and hashCode() methods to the generated file causing the problem. Updated the reply-to cache in line with the 0-8/0-9/0-9-1 model of using a LinkedHashMap which will cache the most recent 500 Destinations rather than just SoftReferencing them all.

          Show
          Robbie Gemmell added a comment - Added a test to verify the issue, and its resolution following addition of equals() and hashCode() methods to the generated file causing the problem. Updated the reply-to cache in line with the 0-8/0-9/0-9-1 model of using a LinkedHashMap which will cache the most recent 500 Destinations rather than just SoftReferencing them all.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development