Qpid
  1. Qpid
  2. QPID-2244

AMQQueueMBean.viewMessages does not show 0-10 protocol MessageTransferMessage's

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.6
    • Fix Version/s: 0.6
    • Component/s: Java Broker
    • Labels:

      Description

      AMQQueueMBean.viewMessages does not show 0-10 protocol MessageTransferMessage's, it can only show messages based on the AMQMessage class used by lower ptorocol versions. As such a user will not see messages on the queue if they are produced with a 0-10 client but will for a 0-8/0-9 client.

        Activity

        Hide
        Marnie McCormack added a comment -

        Comment from Robbie re the status of this JIRA, which I've actioned:

        Martin and I have both made some additions so the 0-10 messages now show up
        in the message list and have similar basic header output to the 0-8/0-9
        messages, so it is no longer in a blocking position. A couple of the values
        reported previously (clusterID and propertyFlags) don't seem to exist/be
        exposed in the 0-10 code so they are just showing with empty values just
        now.

        With that, move 2244 to RTR? (done)

        Show
        Marnie McCormack added a comment - Comment from Robbie re the status of this JIRA, which I've actioned: Martin and I have both made some additions so the 0-10 messages now show up in the message list and have similar basic header output to the 0-8/0-9 messages, so it is no longer in a blocking position. A couple of the values reported previously (clusterID and propertyFlags) don't seem to exist/be exposed in the 0-10 code so they are just showing with empty values just now. With that, move 2244 to RTR? (done)
        Hide
        Marnie McCormack added a comment -

        Changes look ok with caveat that propertyFlags and clusterID not available on 0-10 messages

        Show
        Marnie McCormack added a comment - Changes look ok with caveat that propertyFlags and clusterID not available on 0-10 messages
        Hide
        Rob Godfrey added a comment -

        clusterID never meant anything
        propertyFlags would have been a weird thing to expose directly - it was simply a way of packing multiple binary flags (immediate, etc.) into a single integer to save on memory overhead

        Show
        Rob Godfrey added a comment - clusterID never meant anything propertyFlags would have been a weird thing to expose directly - it was simply a way of packing multiple binary flags (immediate, etc.) into a single integer to save on memory overhead
        Hide
        Robbie Gemmell added a comment -

        Should we instead be removing those from display for the 0-8/0-9(and 0-10) mesages then?

        Show
        Robbie Gemmell added a comment - Should we instead be removing those from display for the 0-8/0-9(and 0-10) mesages then?
        Hide
        Rob Godfrey added a comment -

        Robbie: yeah - I think that would be sensible... the stuff that is held in "propertyFlags" should probably be displayed as the individual component properties however. Definitely not a blocker though

        Show
        Rob Godfrey added a comment - Robbie: yeah - I think that would be sensible... the stuff that is held in "propertyFlags" should probably be displayed as the individual component properties however. Definitely not a blocker though

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development