Qpid
  1. Qpid
  2. QPID-3302

Can't send QMFv2 requests from JMS due to inability to set app-id

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.10
    • Fix Version/s: 0.11
    • Component/s: Java Client
    • Labels:
      None

      Activity

      Gordon Sim created issue -
      Gordon Sim made changes -
      Field Original Value New Value
      Summary Can't sned QMFv2 requests from JMS due to inability to set app-id Can't send QMFv2 requests from JMS due to inability to set app-id
      Hide
      Gordon Sim added a comment -

      This allows the app-id to be set in a manner similar to the messaging API in c++ and python.

      Show
      Gordon Sim added a comment - This allows the app-id to be set in a manner similar to the messaging API in c++ and python.
      Gordon Sim made changes -
      Attachment amqp-0-10_app-id.patch [ 12482331 ]
      Rajith Attapattu made changes -
      Assignee Rajith Attapattu [ rajith ]
      Hide
      Rajith Attapattu added a comment -

      Fixed with a test case.
      It would be nice if we use constants instead of literals like "x-amqp-0-10.app-id".

      I am working on a separate patch to organize all constants used in the code into a single place and also to document them.

      Show
      Rajith Attapattu added a comment - Fixed with a test case. It would be nice if we use constants instead of literals like "x-amqp-0-10.app-id". I am working on a separate patch to organize all constants used in the code into a single place and also to document them.
      Rajith Attapattu made changes -
      Status Open [ 1 ] Resolved [ 5 ]
      Resolution Fixed [ 1 ]
      Hide
      Keith Wall added a comment -

      Hi Rajith,

      This commit broke the default profile build. Please see:

      https://builds.apache.org/view/M-R/view/Qpid/job/qpid-java-build/156/

      Thanks, Keith.

      Show
      Keith Wall added a comment - Hi Rajith, This commit broke the default profile build. Please see: https://builds.apache.org/view/M-R/view/Qpid/job/qpid-java-build/156/ Thanks, Keith.
      Hide
      Rajith Attapattu added a comment -

      Sorry I forgot to commit the change to the exclude file.
      Should be ok now.

      Show
      Rajith Attapattu added a comment - Sorry I forgot to commit the change to the exclude file. Should be ok now.
      Transition Time In Source Status Execution Times Last Executer Last Execution Date
      Open Open Resolved Resolved
      14d 15h 50m 1 Rajith Attapattu 28/Jun/11 04:09

        People

        • Assignee:
          Rajith Attapattu
          Reporter:
          Gordon Sim
        • Votes:
          0 Vote for this issue
          Watchers:
          1 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development