Qpid
  1. Qpid
  2. QPID-4680

[Java Broker] Persistent NoAck messages leave rows in queue entry table even after dequeue (0-9/0-9/0-9-1 codepath)

        Details

        • Type: Bug Bug
        • Status: Resolved
        • Priority: Major Major
        • Resolution: Fixed
        • Affects Version/s: 0.14, 0.16, 0.18, 0.20
        • Fix Version/s: 0.22, 0.23
        • Component/s: Java Broker
        • Labels:
          None

          Description

          SubscriptionImpl.NoAckSubscription is not correctly removing the record of messages from the store when dequing- it is only removing the in memory record.

          In order to remove the store record, a server transaction must be run against the message store in addition to the dequeue on the in-memory queue.

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Rob Godfrey added a comment -

            Committed revision 1462941.

            Show
            Rob Godfrey added a comment - Committed revision 1462941.
            Hide
            Permalink
            Robbie Gemmell added a comment -

            Change looks good to me. As discussed I am going to request this for 0.22.

            Show
            Robbie Gemmell added a comment - Change looks good to me. As discussed I am going to request this for 0.22.
            Hide
            Permalink
            Justin Ross added a comment -

            Reviewed by Robbie. Approved for 0.22.

            Show
            Justin Ross added a comment - Reviewed by Robbie. Approved for 0.22.
            Hide
            Permalink
            Rob Godfrey added a comment -

            Merged to 0.22 as r1463179.

            Show
            Rob Godfrey added a comment - Merged to 0.22 as r1463179.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development