On Fri, Mar 14, 2008 at 7:36 AM, ITAGAKI Takahiro <[EMAIL PROTECTED]> wrote: > > I tried to make a patch to exclude PROC_VACUUM_FOR_WRAPAROUND > from PROC_VACUUM_STATE_MASK and make autovacuum workers to clear > PROC_VACUUM_FOR_WRAPAROUND by themselves. Is it a reasonable solution? > >
Looks good to me. Otherwise we can pass additional parameter to autovacuum_do_vac_analyze() and then use vacstmt to pass the information to vacuum(). Not sure which is a cleaner way though. I also noticed that inside autovacuum_do_vac_analyze(), we save the old context (which is TopTransactionContext) and restore it back after vacuum() returns. But vacuum() might have started a new transaction invalidating the saved context. Do we see any problem here ? Thanks, Pavan -- Pavan Deolasee EnterpriseDB http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
