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

      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.
      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 -

      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.
      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.

        People

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

          Dates

          • Created:
            Updated:
            Resolved:

            Development