Bruce Momjian wrote:
> Alvaro has just applied a modified version of this patch.

Hannu, I'm curious:

> Hannu Krosing wrote:

> > Ok, this is a new version of the vacuum patch with the following changes
> > following some suggestions in this thread.
> > 
> > * changed the patch to affect only lazy vacuum 
> > * moved inVacuum handling to use PG_TRY
> > * moved vac_update_relstats() out of lazy_vacuum_rel into a separate
> >   transaction. The code to do this may not be the prettiest, maybe it
> >   should use a separate struct.

What was idea behind moving vac_update_relstats to a separate
transaction?  I'm wondering if it's still needed, if it further enhances
the system somehow, or your patch did something differently than what
was applied.

Alvaro Herrera                      
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend

Reply via email to