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 |
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 |