Re: Improvement of procArray.xmin for VACUUM - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: Improvement of procArray.xmin for VACUUM
Date
Msg-id 200703270211.l2R2BWS09883@momjian.us
Whole thread Raw
In response to Re: Improvement of procArray.xmin for VACUUM  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Improvement of procArray.xmin for VACUUM
List pgsql-patches
Tom Lane wrote:
> Bruce Momjian <bruce@momjian.us> writes:
> > Tom Lane wrote:
> >> ... but we won't come out ahead unless advancing xmin
> >> intra-transaction really helps, and I'm not sure I believe that (except
> >> in the special case of VACUUM, and we already have a solution for that,
> >> which would be independent of this).
>
> > The improvement is going to be a win for multi-statement transactions
> > --- the only question is how often are they long-running.
>
> Uh, no, that's not very clear.  A long-running transaction will be a
> VACUUM bottleneck because of its own XID, never mind its xmin.  To make
> this helpful, you have to posit a lot of overlapping long-running
> transactions (such that the distance back to GlobalXmin might average
> about twice the distance back to the oldest live XID).  That's not
> impossible but I wonder whether it's not mostly a token of bad
> application design.

Well, my secondary addition was to start MyProc->xmin with the current
xid counter, rather than your own xid.  This is because your xid is
already in progress and so will not be touched, so why not start with
the current xid counter.

--
  Bruce Momjian  <bruce@momjian.us>          http://momjian.us
  EnterpriseDB                               http://www.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: patch adding new regexp functions
Next
From: Bruce Momjian
Date:
Subject: Re: Improvement of procArray.xmin for VACUUM