Qpid
  1. Qpid
  2. QPID-831

[Java Broker] Possible message loss on requeuing of messages on shared queues

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: M2.1
    • Fix Version/s: M2.1
    • Component/s: Java Broker
    • Labels:
      None

      Description

      Changes introduced in QPID-711 to introduce QueueEntry class made worse an existing "misuse" of getDeliveredToConsumer a property of the message. Earlier changes for putting messages on PDQs had used this property as a proxy for whether the queue entry had already been delievered. This is an incorrect usage when a Message has been delivered to multiple queues.

      Fix is to remove the attempted shortcut logic for delivering to PDQs; also use taken( ) rather than getDTC for adjusting queue size on message expiration.

        Activity

        Rob Godfrey created issue -
        Hide
        Marnie McCormack added a comment -

        Required changes committed & ok (see svn comments)

        Show
        Marnie McCormack added a comment - Required changes committed & ok (see svn comments)
        Marnie McCormack made changes -
        Field Original Value New Value
        Assignee Rob Godfrey [ rgodfrey ]
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Gavin made changes -
        Workflow jira [ 12425060 ] QPid [ 12438667 ]
        Gavin made changes -
        Workflow QPid [ 12438667 ] QPid Workflow [ 12439950 ]
        Rob Godfrey made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        3d 24m 1 Marnie McCormack 07/Mar/08 13:24
        Resolved Resolved Closed Closed
        2532d 6h 42m 1 Rob Godfrey 11/Feb/15 20:06

          People

          • Assignee:
            Rob Godfrey
            Reporter:
            Rob Godfrey
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development