[QPID-1764] Create Abstract base class to perform new reference counting for TransactionLog

• Type: Bug
• Status: Closed
• Priority: Blocker
• Resolution: Won't Fix
• Fix Version/s: 0.6
• Assignee: Aidan Skinner
• Reporter: Martin Ritchie
• Created: 20/Mar/09 14:49
• Updated: 18/Sep/12 22:34
• Resolved: 28/Oct/09 01:53

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