Re: [v9.2] DROP Reworks Part.1 - Consolidate routines to handle DropStmt
Robert Haas <robertmhaas@gmail.com>
From: Robert Haas <robertmhaas@gmail.com>
To: Alvaro Herrera <alvherre@commandprompt.com>
Cc: Kohei KaiGai <kaigai@kaigai.gr.jp>,
PgHacker <pgsql-hackers@postgresql.org>
Date: 2011-07-06T18:02:13Z
Lists: pgsql-hackers
On Wed, Jul 6, 2011 at 1:09 PM, Alvaro Herrera <alvherre@commandprompt.com> wrote: > Excerpts from Robert Haas's message of mié jul 06 12:40:39 -0400 2011: > >> This patch removes an impressive amount of boilerplate code and >> replaces it with something much more compact. I like that. In the >> interest of full disclosure, I suggested this approach to KaiGai at >> PGCon, so I'm biased: but even so, I'm pleasantly surprised by the >> amount of consolidation that appears possible here. > > Yeah. Myself, I love the fact that the dropmsgstrings thing is gone. I > wonder if the routine to obtain "foo doesn't exist, skipping" messages > could be replaced by judicious use of getObjectDescription. I've been told we don't want to go further in that direction for reasons of translatability. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company