Re: autovauum integration patch: Attempt #4

From: "Matthew T(dot) O'Connor" <matthew(at)zeut(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org>, Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Subject: Re: autovauum integration patch: Attempt #4
Date: 2004-07-24 06:59:51
Message-ID: 1090652391.19256.10.camel@zedora2
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

On Fri, 2004-07-23 at 23:25, Tom Lane wrote:
> "Matthew T. O'Connor" <matthew(at)zeut(dot)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 Content-Type Size
pg_autovacuum.h text/x-chdr 3.9 KB
diff text/x-patch 77.2 KB

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2004-07-24 07:01:34 Re: autovauum integration patch: Attempt #4
Previous Message Tom Lane 2004-07-24 03:25:24 Re: autovauum integration patch: Attempt #4