| From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
|---|---|
| To: | Dimitri Fontaine <dimitri(at)2ndquadrant(dot)fr> |
| Cc: | Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: ALTER OBJECT any_name SET SCHEMA name |
| Date: | 2010-11-04 14:39:18 |
| Message-ID: | 1288881383-sup-5220@alvh.no-ip.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Excerpts from Dimitri Fontaine's message of jue nov 04 11:06:48 -0300 2010:
> Also attached, please find the complete version of ALTER EXTENSION ext
> SET SCHEMA name; with support for all contrib extensions. That's the
> example that allows to see the API (AlterFooNamespace_oid and _internal
> functions) in action: that should help devising the best refactoring.
Three comments,
1. wouldn't it make more sense to save the extension namespace in the
extension catalog?
2. I think the guts of AlterExtensionNamespace (the large switch block)
should be elsewhere, probably in alter.c
3. Not this patch, but I think using "extension" as a global variable
name is a bad idea.
--
Álvaro Herrera <alvherre(at)commandprompt(dot)com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2010-11-04 14:51:08 | Re: ALTER OBJECT any_name SET SCHEMA name |
| Previous Message | Tom Lane | 2010-11-04 14:39:16 | Re: Alter column to type serial |