Re: autovacuum integration attempt #3 - Mailing list pgsql-patches

From Matthew T. O'Connor
Subject Re: autovacuum integration attempt #3
Date
Msg-id 23596.68.175.84.212.1090447133.squirrel@68.175.84.212
Whole thread Raw
In response to Re: autovacuum integration attempt #3  (Bruce Momjian <pgman@candle.pha.pa.us>)
Responses Re: autovacuum integration attempt #3  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-patches
Sorry, perhaps my initial instructions weren't clear.

You have to move the pg_autovacuum.[ch] files then apply the patch.  The
patch alters them considerably.  Should I submit the patch in a different
way?

Thanks,

Matthew


>
> I am having problem applying this.  I don't think we can just move
> pg_autovacuum.[ch] right into the backend code.  There are issues with
> include files and there is no definition for autovacuum_vacuum_base and
> other variables, and there is no AutoVacMain().  I can try guessing on
> those but I think I need a more complete patch or more complete
> instructions to get this applied.  Also, it uses the gcc-ism
> __attribute__ that isn't portable to other compilers.
>
> ---------------------------------------------------------------------------
>
> Matthew T. O'Connor wrote:
>> Ok, here is a new patch for to integrate pg_autovacuum into the backend.
>>
>> This patch adds the following to that last patch:
>> * updated to latest CVS
>> * changed GUC vars from autovac_* to autovacuum_*
>> * changed autovac_enabled GUC variable to autovacuum
>> * changed autovacuum GUC variable default to false
>> * improved autovacuum failure when row level stats aren't enabled
>>
>> As before, this patch requires pg_autovacuum.c and .h get moved
>> from contrib to src/backend/postmaster and src/include/postmaster
>> respectively. In addition, the attached pg_autovacuum.h file must be put
>> in src/include/catelog/ for the new pg_autovacuum system
>> table.
>>
>> Please apply to CVS or tell me what I need to change to get it applied.
>
> --
>   Bruce Momjian                        |  http://candle.pha.pa.us
>   pgman@candle.pha.pa.us               |  (610) 359-1001
>   +  If your life is a hard drive,     |  13 Roberts Road
>   +  Christ can be your backup.        |  Newtown Square, Pennsylvania
> 19073
>
>


pgsql-patches by date:

Previous
From: Mark Kirkwood
Date:
Subject: Re: win32 readline
Next
From: Bruce Momjian
Date:
Subject: Re: autovacuum integration attempt #3