Re: Async Commit, v21 (now: v22)

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