Re: autovauum integration patch: Attempt #4 - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: autovauum integration patch: Attempt #4
Date
Msg-id 200407240701.i6O71Yq22329@candle.pha.pa.us
Whole thread Raw
In response to Re: autovauum integration patch: Attempt #4  ("Matthew T. O'Connor" <matthew@zeut.net>)
Responses Re: autovauum integration patch: Attempt #4  (Bruce Momjian <pgman@candle.pha.pa.us>)
Re: autovauum integration patch: Attempt #4  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-patches
Patch applied.  Thanks.

---------------------------------------------------------------------------


Matthew T. O'Connor wrote:
> On Fri, 2004-07-23 at 23:25, Tom Lane wrote:
> > "Matthew T. O'Connor" <matthew@zeut.net> writes:
> > > Peter Eisentraut wrote:
> > >> I'm not sure whether we can allow int8 columns in system tables, for
> > >> portability reasons.
> >
> > > Someone will have to tell me if this is really a problem,
> >
> > Depending on int8 to actually work is right out.
> >
> > I'd go with float or double depending on your precision needs.
>
> Ok, I went with float.
>
> Also, I changed pg_autovacuum.[ch] to autovacuum.[ch] as Peter
> suggested. So now to apply this patch:
>
> 1) Move pg_autovacuum.[ch] from contrib to
> src/backend/postmaster/autovacuum.c and
> src/include/postmaster/autovacuum.h respectively.
> 2) Place the attached pg_autovacuum.h in src/include/catelog/
> 3) Apply the attached diff file
>
> Please apply to CVS or tell me what I need to change to get it applied.
>
> Thanks again,
>
>
> Matthew
>
>

[ Attachment, skipping... ]

[ Attachment, skipping... ]

--
  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: "Matthew T. O'Connor"
Date:
Subject: Re: autovauum integration patch: Attempt #4
Next
From: Bruce Momjian
Date:
Subject: Re: autovauum integration patch: Attempt #4