Re: [PATCHES] [DOCS] rename of a view

2007-07-03 Thread Neil Conway
On Tue, 2007-07-03 at 09:56 -0400, Tom Lane wrote: > Neil, according to > http://developer.postgresql.org/index.php/Todo:PatchStatus > you have accepted two patches to review --- what is happening with > those? Yeah, sorry, forgot about those -- I'll take a look at them tonight. -Neil

Re: [PATCHES] [DOCS] rename of a view

2007-07-03 Thread Tom Lane
Stefan Kaltenbrunner <[EMAIL PROTECTED]> writes: > Neil Conway wrote: >> On Mon, 2007-02-07 at 23:13 -0400, Tom Lane wrote: >>> Er, was this on the agenda for 8.3? >> >> Well, it seemed fairly harmless to me (no behavioral changes and very >> little new code, just syntax), so I didn't see a compel

Re: [PATCHES] [DOCS] rename of a view

2007-07-02 Thread Stefan Kaltenbrunner
Neil Conway wrote: > On Mon, 2007-02-07 at 23:13 -0400, Tom Lane wrote: >> Er, was this on the agenda for 8.3? > > Well, it seemed fairly harmless to me (no behavioral changes and very > little new code, just syntax), so I didn't see a compelling reason to > delay applying it for a few months. But

Re: [PATCHES] [DOCS] rename of a view

2007-07-02 Thread Neil Conway
On Mon, 2007-02-07 at 23:13 -0400, Tom Lane wrote: > Er, was this on the agenda for 8.3? Well, it seemed fairly harmless to me (no behavioral changes and very little new code, just syntax), so I didn't see a compelling reason to delay applying it for a few months. But I can revert it if you'd pref

Re: [PATCHES] [DOCS] rename of a view

2007-07-02 Thread Tom Lane
Neil Conway <[EMAIL PROTECTED]> writes: > On Sun, 2007-07-01 at 12:55 -0700, David Fetter wrote: >> Here's a new patch + file. This one allows ALTER [SEQUENCE | VIEW] to >> work only on the respective database objects, but permits the old >> ALTER TABLE syntax. > Applied with some fixes. Thanks f

Re: [PATCHES] [DOCS] rename of a view

2007-07-02 Thread Neil Conway
On Sun, 2007-07-01 at 12:55 -0700, David Fetter wrote: > Here's a new patch + file. This one allows ALTER [SEQUENCE | VIEW] to > work only on the respective database objects, but permits the old > ALTER TABLE syntax. Applied with some fixes. Thanks for the patch. ALTER VIEW ... SET SCHEMA might b

Re: [PATCHES] [DOCS] rename of a view

2007-07-02 Thread David Fetter
On Sun, Jul 01, 2007 at 09:03:45PM -0700, Neil Conway wrote: > On Sun, 2007-01-07 at 12:55 -0700, David Fetter wrote: > > Here's a new patch + file. This one allows ALTER [SEQUENCE | VIEW] to > > work only on the respective database objects, but permits the old > > ALTER TABLE syntax. > > How abo

Re: [PATCHES] [DOCS] rename of a view

2007-07-01 Thread Neil Conway
On Sun, 2007-01-07 at 12:55 -0700, David Fetter wrote: > Here's a new patch + file. This one allows ALTER [SEQUENCE | VIEW] to > work only on the respective database objects, but permits the old > ALTER TABLE syntax. How about taking a look at the more thorough documentation updates Tom suggested

Re: [PATCHES] [DOCS] rename of a view

2007-07-01 Thread David Fetter
On Sat, Jun 30, 2007 at 01:48:51AM -0400, Tom Lane wrote: > Neil Conway <[EMAIL PROTECTED]> writes: > > For the ALTER SEQUENCE case, I think it's also a little weird to > > have an ALTER SEQUENCE command that modifies some of the > > properties of a sequence, but not the sequence's name. While that