Re: enable/disable trigger (Re: Fwd: [HACKERS] Open items)

Поиск
Список
Период
Сортировка
От Satoshi Nagayasu
Тема Re: enable/disable trigger (Re: Fwd: [HACKERS] Open items)
Дата
Msg-id 42F6E9F8.9070709@nttdata.co.jp
обсуждение исходный текст
Ответ на Re: enable/disable trigger (Re: Fwd: [HACKERS] Open items)  (Alvaro Herrera <alvherre@alvh.no-ip.org>)
Ответы Re: enable/disable trigger (Re: Fwd: [HACKERS] Open items)  (Alvaro Herrera <alvherre@alvh.no-ip.org>)
Список pgsql-patches
Alvaro Herrera wrote:
> There are a few elog() calls that should really be ereport().  Also
> this message

I've fixed to call ereport() on permission error.

>>+     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)

What does "really be two messages" mean?

> There's a SQL typo here:
>
>>+             appendPQExpBuffer(query, "ALTER TABLE %s DIABLE TRIGGER %s;\n",

Fixed.

--
NAGAYASU Satoshi <nagayasus@nttdata.co.jp>

В списке pgsql-patches по дате отправления:

Предыдущее
От: Alvaro Herrera
Дата:
Сообщение: Re: enable/disable trigger (Re: Fwd: [HACKERS] Open items)
Следующее
От: Brendan Jurd
Дата:
Сообщение: Re: gettime() - a timeofday() alternative