"Pavan Deolasee" <[EMAIL PROTECTED]> writes:
> On Wed, Mar 12, 2008 at 7:13 PM, Heikki Linnakangas
> <[EMAIL PROTECTED]> wrote:
>> Yep, patch attached. I also changed xactGetCommittedChildren to return
>> the original array instead of copying it, as Alvaro suggested.
> Any comments on the flag based approach I suggested earlier ?
I didn't like it; it seemed overly complicated (consider dealing with
XID wraparound), and it would have problems with a slow transaction
generating a sparse set of subtransaction XIDs. I think getting rid of
the linear search will be enough to fix the performance problem.
regards, tom lane
--
Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-patches
