On Wed, 2007-08-01 at 18:54 -0400, Tom Lane wrote:
> "Simon Riggs" <simon@2ndquadrant.com> writes:
> > Here's v23, including all suggested changes, plus some reworking of the
> > transaction APIs to reduce the footprint of the patch.
>
> Applied with some editorialization ---
Thanks
> I found a few bugs, as well as
> things that seemed they could be neater.
Keen to improve: any chance of explaining, possibly offlist?
> Notable was that the patch
> would have significantly increased the contention for CLogControlLock by
> requiring two lock cycles to fetch a transaction's commit status and
> LSN. I changed it so that TransactionIdGetStatus returns both pieces
> of information.
Thanks for picking that up.
--
Simon Riggs
EnterpriseDB http://www.enterprisedb.com