Re: ALTER OBJECT any_name SET SCHEMA name
David Fetter <david@fetter.org>
From: David Fetter <david@fetter.org>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Dimitri Fontaine <dimitri@2ndquadrant.fr>, Tom Lane <tgl@sss.pgh.pa.us>, Alvaro Herrera <alvherre@commandprompt.com>, Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>, PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2010-11-21T18:03:17Z
Lists: pgsql-hackers
On Sun, Nov 21, 2010 at 07:53:57AM -0500, Robert Haas wrote: > On Sat, Nov 20, 2010 at 11:23 PM, Robert Haas <robertmhaas@gmail.com> wrote: > > Ah, nuts. I see now there's a v7. Never mind... > > OK. I looked at the right version, now. Hopefully. > > It seems we have no regression tests at all for any of the existing > SET SCHEMA commands. This seems like a good time to correct that > oversight, and also add some for the new commands you're adding here. > (It might be helpful to submit the regression tests for the existing > commands as a separate patch.) Also, you're missing psql tab > completion support, which would be nice to have. > > In CheckSetNamespace() you have the message 'already exists in schema' > there where the existing, similar checks say 'is already in schema', > which is a bit confusing. But that code looks useful, and in fact I > think we should use it for the existing object types also to avoid > code duplication. Should this really error out? It's just a NOOP, so perhaps a NOTICE would be more appropriate. Cheers, David. -- David Fetter <david@fetter.org> http://fetter.org/ Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter Skype: davidfetter XMPP: david.fetter@gmail.com iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics Remember to vote! Consider donating to Postgres: http://www.postgresql.org/about/donate