From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Neil Conway <neilc(at)samurai(dot)com> |
Subject: | Re: Open items list for 8.1 |
Date: | 2005-09-28 01:58:01 |
Message-ID: | 200509280158.j8S1w1D25943@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Tom Lane wrote:
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > fix ALTER SCHEMA RENAME for sequence dependency, or remove feature
>
> I've posted a proposed patch to fix this. The patch requires an initdb
> (to add new sequence functions), so if we do that we may as well also
> fix the 32/64bit risk mentioned here:
> http://archives.postgresql.org/pgsql-hackers/2005-09/msg01241.php
>
> Also, the floor seems open to discuss whether or not to revert the file
> access functions to their pre-beta2 APIs. I've got mixed feelings about
> that myself, but you can certainly make a case that the current
> definitions are not enough cleaner than what was there before to justify
> changing. This seems particularly true for pg_cancel_backend(), which
> already was in the core in 8.0.
I am thinking we should keep things as they are now.
I remember two changes of significance. First, pg_cancel_backend()'s
return value was change to boolean. I think the compelling argument
here is that we are adding other functions that _should_ return boolean,
and to keep pg_cancel_backend() as 1/0 was kind of strange. Also, I
assume pg_cancel_backend() is not a general use function and therefore
is more of an admin function that we can adjust as needed to improve the
API. We have always allowed rare/admin functions to be improved without
as much concern for backward compatibility as a more mainstream feature.
The other change was the rename of pg_complete_relation_size() to
pg_total_relation_size(). While there was a huge (exhausting)
discussion that finalized on pg_complete_relation_size(), a number of
people felt pg_total_relation_size() was better.
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2005-09-28 02:38:50 | Re: Proposed patch for sequence-renaming problems |
Previous Message | Jim C. Nasby | 2005-09-28 00:22:03 | Re: Vacuum questions... |