Uploaded image for project: 'Qpid'
  1. Qpid
  2. QPID-4512

[Java Broker] JMX QueueMBean#deleteMessages only deletes the first matching message from the range

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.18, 0.20
    • Fix Version/s: 0.20
    • Component/s: Java Broker
    • Labels:
      None

      Description

      Upon trying to delete the range of messages from the queue using JMX Queue MBean deleteMessages operation, only the first matching message from the range is deleted and the rest is simply ignored.

        Activity

        Hide
        gemmellr Robbie Gemmell added a comment -

        Fix committed in http://svn.apache.org/viewvc?rev=1424757&view=rev

        Alex, can you review this change please? I would like to request it for 0.20 if there is time, given that it is a highly isolated and rather trivial change.

        Show
        gemmellr Robbie Gemmell added a comment - Fix committed in http://svn.apache.org/viewvc?rev=1424757&view=rev Alex, can you review this change please? I would like to request it for 0.20 if there is time, given that it is a highly isolated and rather trivial change.
        Hide
        alex.rufous Alex Rudyy added a comment -

        The committed changes look good for me.

        I agree that they are isolated and trivial. Their inclusion into 0.20 will allow the java broker users to delete messages via JMX.

        Show
        alex.rufous Alex Rudyy added a comment - The committed changes look good for me. I agree that they are isolated and trivial. Their inclusion into 0.20 will allow the java broker users to delete messages via JMX.
        Hide
        justi9 Justin Ross added a comment -

        Reviewed by Alex. Approved for 0.20.

        Show
        justi9 Justin Ross added a comment - Reviewed by Alex. Approved for 0.20.
        Hide
        gemmellr Robbie Gemmell added a comment -

        Now merged to the 0.20 release branch.

        Show
        gemmellr Robbie Gemmell added a comment - Now merged to the 0.20 release branch.

          People

          • Assignee:
            alex.rufous Alex Rudyy
            Reporter:
            alex.rufous Alex Rudyy
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development