On Thursday 14 July 2005 09:23, Christopher Browne wrote:
> Darcy Buskermolen wrote:
> >Hello,
> >
> >I have implemented an experimental patch that avoids index maintainance
> > during the SUBSCRIBE SET phase of replication.  This work is based apon
> > theory and findings of Jan.   My initial testing shows that  subscribing
> > a large heavily hit pgbench (scalling factor of 100 with 50 concurrent
> > clients) database can subscribe set in the same amount of time as it
> > takes to pg_dump | psql.
> >
> >Those of you who are interested can find the patch for 1.1 and CVS HEAD at
> >http://www.dbitech.ca/slony/patches/subscription-perf-patch . I make no
> > firm claim that, at this time, this patch will act as intended for
> > everybody.  It would help us in getting this patch fully ironed out and
> > applied to the main tree, if those of you who do try this patch would
> > report your findings back to the list, wether it be yes it works or I got
> > an error.
>
> It didn't work as provided; the 'oid' reference in the second line of
> the query was ambiguous.
>
> Use 'where pg_catalog.pg_class.oid' instead of 'where oid' in the two
> updates on pg_catalog.pg_class, and that turns out better...
>
> +#if 1
> +                 "update pg_catalog.pg_class set relhasindex = 'f'
> FROM  %s.sl_table "
> +                 "where pg_catalog.pg_class.oid = %s.sl_table.tab_reloid "
> +                 "and %s.slon_quote_input('\"' ||
> %s.sl_table.tab_nspname || '\".\"' || %s.sl_table.tab_relname || '\"') =
> '%s'; "
> +#endif

Good catch Christopher.   I have updated the patch to reflect this change.

-- 
Darcy Buskermolen
Wavefire Technologies Corp.

http://www.wavefire.com
ph: 250.717.0200
fx: 250.763.1759