Re: [HACKERS] statement level trigger causes pltcl, plpython SIGSEGV

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: [HACKERS] statement level trigger causes pltcl, plpython SIGSEGV
Дата
Msg-id 8786.1060022603@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: [HACKERS] statement level trigger causes pltcl, plpython  (Joe Conway <mail@joeconway.com>)
Ответы Re: [HACKERS] statement level trigger causes pltcl, plpython  (Joe Conway <mail@joeconway.com>)
Список pgsql-patches
Joe Conway <mail@joeconway.com> writes:
> This patch includes pltcl and plpython, with the mentioned style issue
> fixed. Both PLs pass their scripted tests and my simple statement level
> trigger test.

Applied, thanks.

BTW, one other stylistic nit: I don't think comments like

        /* internal error */
        elog(ERROR, "unrecognized OP tg_event: %u", tdata->tg_event);

are really necessary.  In the brave new ereport world, any elog(ERROR)
call is an internal error by definition --- if it isn't, you should be
using ereport.  So the use of elog is sufficient documentation.  IMHO
anyway.

            regards, tom lane

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

Предыдущее
От: Joe Conway
Дата:
Сообщение: Re: [HACKERS] statement level trigger causes pltcl, plpython
Следующее
От: Joe Conway
Дата:
Сообщение: Re: [HACKERS] statement level trigger causes pltcl, plpython