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

        Hiram Chirino created issue -
        Hiram Chirino made changes -
        Field Original Value New Value
        Component/s Java Client [ 12311389 ]
        Rob Godfrey made changes -
        Summary getJMSReplyTo() != the value set with setJMSReplyTo() [AMQP 1.0 JMS Client] getJMSReplyTo() != the value set with setJMSReplyTo()
        Hide
        Hiram Chirino added a comment -

        Attaching patch which implements a fix.

        Show
        Hiram Chirino added a comment - Attaching patch which implements a fix.
        Hiram Chirino made changes -
        Attachment QPID-4454.patch [ 12555547 ]
        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.
        Hiram Chirino made changes -
        Attachment QPID-4454-v2.patch [ 12555559 ]
        Hiram Chirino made changes -
        Attachment QPID-4454.patch [ 12555547 ]
        Rob Godfrey made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.21 [ 12323549 ]
        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
        Justin Ross made changes -
        Assignee Rob Godfrey [ rgodfrey ]
        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.
        Justin Ross made changes -
        Fix Version/s 0.20 [ 12323548 ]
        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