Qpid
  1. Qpid
  2. QPID-1764

Create Abstract base class to perform new reference counting for TransactionLog

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: 0.6
    • Component/s: None
    • Labels:
      None

      Description

      Summary:

      Summary the introduction of the new TransactionLog interface over the old MessageStore has resulted in each of the old Stores performing their own reference counting. As a result they all suffer from:
      a) Being different implementations
      b) Not being fully tested by existing test frameworks.

      By creating a single reference counting TransactionLog base class that delegates the simple storage and retreival tasks to the underlying Stores the stores can be simpler and the testing more complete.

      Current testing has already shown that performance has knocked 5% by the MemoryMessageStore's implementation of reference counting when it should not need to be doing any reference counting.

      This was brought up during the FtD Code review
      http://cwiki.apache.org/confluence/display/qpid/FtD+Code+Review+Notes

        Activity

        No work has yet been logged on this issue.

          People

          • Assignee:
            Aidan Skinner
            Reporter:
            Martin Ritchie
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development