Qpid
  1. Qpid
  2. QPID-4379

HA does not properly handle expired messages

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.18
    • Fix Version/s: 0.19
    • Component/s: C++ Clustering
    • Labels:
      None

      Description

      Description of problem:
      If a message expires in a queue before the HA replicator is able to browse the message, the message will be skipped and not replicated. This results in the expired message's async completion never being finalized which can stall the original source of the expired message since it will never receive a completion.

      Oct 18 10:56:02 itcm24 qpidd[48819]: 2012-10-18 10:56:02 [Broker] debug Browser skipping message from 'QueueXyz'

      Version-Release number of selected component (if applicable):
      Qpid 0.18

      How reproducible:
      Frequently

      Steps to Reproduce:
      1. It's a race condition between a message expiring in the queue and the HA browsing subscription being able to replicate it.

      Actual results:
      The expired message is skipped and its async completion is never finalized.

      Expected results:
      While it is not truly necessary for the HA replicating subscription to replicate an expired message, the async completion needs to be finished.

      Additional info:

      see also https://bugzilla.redhat.com/show_bug.cgi?id=868360

        Activity

        Justin Ross made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Justin Ross made changes -
        Fix Version/s 0.19 [ 12322452 ]
        Alan Conway made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Alan Conway added a comment -

        The trunk is not affected by this bug, so this commit is just to add the regression test that proves it.

        committed r1400199 on trunk

        Show
        Alan Conway added a comment - The trunk is not affected by this bug, so this commit is just to add the regression test that proves it. committed r1400199 on trunk
        Alan Conway created issue -

          People

          • Assignee:
            Alan Conway
            Reporter:
            Alan Conway
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development