Qpid
  1. Qpid
  2. QPID-4454

[AMQP 1.0 JMS Client] getJMSReplyTo() != the value set with setJMSReplyTo()

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.20, 0.21
    • Component/s: Java Client
    • Labels:
      None

      Description

      The following test case fails:

      Queue dest = session.createQueue("test");
      Message message = session.createMessage();
      message.setJMSReplyTo(dest);
      assertTrue(message.getJMSReplyTo().equals(dest));

      1. QPID-4454-v2.patch
        22 kB
        Hiram Chirino

        Activity

        Hide
        Hiram Chirino added a comment -

        Attaching patch which implements a fix.

        Show
        Hiram Chirino added a comment - Attaching patch which implements a fix.
        Hide
        Hiram Chirino added a comment -

        Updating patch. Was missing a new file.

        Show
        Hiram Chirino added a comment - Updating patch. Was missing a new file.
        Hide
        Rob Godfrey added a comment -

        Patch applied (with small modification to make checking of annotation attributes work on presence of attributes rather than set equality)

        Revision 1417368: http://svn.apache.org/viewvc?rev=1417368&view=rev

        Show
        Rob Godfrey added a comment - Patch applied (with small modification to make checking of annotation attributes work on presence of attributes rather than set equality) Revision 1417368: http://svn.apache.org/viewvc?rev=1417368&view=rev
        Hide
        Justin Ross added a comment -

        Rob, should we port this one to 0.20?

        Show
        Justin Ross added a comment - Rob, should we port this one to 0.20?
        Hide
        Rob Godfrey added a comment -

        Yes - I'm happy for this to go into 0.20 (the scope is limited to the JMS client and the changes are required to fully implement features of the JMS API)

        Show
        Rob Godfrey added a comment - Yes - I'm happy for this to go into 0.20 (the scope is limited to the JMS client and the changes are required to fully implement features of the JMS API)
        Hide
        Justin Ross added a comment -

        Reviewed by Rob. Approved for 0.20.

        Show
        Justin Ross added a comment - Reviewed by Rob. Approved for 0.20.
        Hide
        Rob Godfrey added a comment -

        Merged to 0.20 branch as revision 1418220.

        Show
        Rob Godfrey added a comment - Merged to 0.20 branch as revision 1418220.

          People

          • Assignee:
            Rob Godfrey
            Reporter:
            Hiram Chirino
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development