On Mon, Aug 08, 2005 at 08:07:05AM +0900, Satoshi Nagayasu wrote:
> Here is new patch with pg_dump modification.

There are a few elog() calls that should really be ereport().  Also
this message

> +     elog(NOTICE, "%d trigger(s) on %s %s.",
> +              changed,
> +              NameStr(rel->rd_rel->relname),
> +              enable ? "enabled" : "disabled");

should really be two messages (Maybe even four: disabled-plural,
disabled-singular, enabled-plural, enabled-singular)


There's a SQL typo here:

> +             appendPQExpBuffer(query, "ALTER TABLE %s DIABLE TRIGGER %s;\n",

-- 
Alvaro Herrera (<alvherre[a]alvh.no-ip.org>)
"La conclusiĆ³n que podemos sacar de esos estudios es que
no podemos sacar ninguna conclusiĆ³n de ellos" (Tanenbaum)

---------------------------(end of broadcast)---------------------------
TIP 4: Have you searched our list archives?

               http://archives.postgresql.org

Reply via email to