Re: bugfix: --echo-hidden is not supported by \sf statements

Поиск
Список
Период
Сортировка
От Pavel Stehule
Тема Re: bugfix: --echo-hidden is not supported by \sf statements
Дата
Msg-id CAFj8pRB3-Tov5s2dCGshp+Vedyk9s97d7hn7rDMmw9Ztrj-7tg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: bugfix: --echo-hidden is not supported by \sf statements  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: bugfix: --echo-hidden is not supported by \sf statements  (Josh Kupershmidt <schmiddy@gmail.com>)
Список pgsql-hackers
2013/1/14 Tom Lane <tgl@sss.pgh.pa.us>:
> Robert Haas <robertmhaas@gmail.com> writes:
>> On Mon, Jan 14, 2013 at 11:56 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>>> So far as I can tell, get_create_function_cmd (and lookup_function_oid
>>> too) were intentionally designed to not show their queries, and for that
>>> matter they go out of their way to produce terse error output if they
>>> fail.  I'm not sure why we should revisit that choice.  In any case
>>> it seems silly to change one and not the other.
>
>> Agreed on the second point, but I think I worked on that patch, and I
>> don't think that was intentional on my part.  You worked on it, too,
>> IIRC, so I guess you'll have to comment on your intentions.
>
>> Personally I think -E is one of psql's finer features, so +1 from me
>> for making it apply across the board.
>
> Well, fine, but then it should fix both of them and remove
> minimal_error_message altogether.  I would however suggest eyeballing
> what happens when you try "\ef nosuchfunction" (with or without -E).
> I'm pretty sure that the reason for the special error handling in these
> functions is that the default error reporting was unpleasant for this
> use-case.

so I rewrote patch

still is simple

On the end I didn't use PSQLexec - just write hidden queries directly
from related functions - it is less invasive

Regards

Pavel

>
>                         regards, tom lane

Вложения

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

Предыдущее
От: Bruce Momjian
Дата:
Сообщение: Re: pg_ctl idempotent option
Следующее
От: Brar Piening
Дата:
Сообщение: Re: Visual Studio 2012 RC