Qpid
  1. Qpid
  2. QPID-4512

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

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major 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

        Alex Rudyy created issue -
        Robbie Gemmell made changes -
        Field Original Value New Value
        Assignee Robbie Gemmell [ gemmellr ]
        Robbie Gemmell made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Robbie Gemmell made changes -
        Status In Progress [ 3 ] Ready To Review [ 10006 ]
        Hide
        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
        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.
        Robbie Gemmell made changes -
        Assignee Robbie Gemmell [ gemmellr ] Alex Rudyy [ alex.rufous ]
        Hide
        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 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.
        Alex Rudyy made changes -
        Status Ready To Review [ 10006 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Robbie Gemmell made changes -
        Summary [Java Broker] JMX QueueMBean#deleteMessages only deletes the first message from the range [Java Broker] JMX QueueMBean#deleteMessages only deletes the first matching message from the range
        Robbie Gemmell made changes -
        Description Upon trying to delete the range of messages from the queue using JMX Queue MBean deleteMessages operation, only the first message from the range is deleted and the rest is simply ignored. 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.
        Robbie Gemmell made changes -
        Fix Version/s 0.21 [ 12323549 ]
        Hide
        Justin Ross added a comment -

        Reviewed by Alex. Approved for 0.20.

        Show
        Justin Ross added a comment - Reviewed by Alex. Approved for 0.20.
        Robbie Gemmell made changes -
        Fix Version/s 0.20 [ 12323548 ]
        Fix Version/s 0.21 [ 12323549 ]
        Hide
        Robbie Gemmell added a comment -

        Now merged to the 0.20 release branch.

        Show
        Robbie Gemmell added a comment - Now merged to the 0.20 release branch.
        Rob Godfrey made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        1d 10h 28m 1 Robbie Gemmell 20/Dec/12 23:25
        In Progress In Progress Reviewable Reviewable
        3m 51s 1 Robbie Gemmell 20/Dec/12 23:29
        Reviewable Reviewable Resolved Resolved
        20m 54s 1 Alex Rudyy 20/Dec/12 23:50
        Resolved Resolved Closed Closed
        782d 20h 16m 1 Rob Godfrey 11/Feb/15 20:07

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development