Re: Proposed p.tch for error locations

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: Proposed p.tch for error locations
Дата
Msg-id 10458.1142285318@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: Proposed p.tch for error locations  (Alvaro Herrera <alvherre@commandprompt.com>)
Список pgsql-patches
I took another look at this and realized that for PQerrorMessage to emit
a cursor-pointer line, it'd be necessary for libpq to hold onto a copy
of the query last sent, which it doesn't do presently.  This is annoying
for long queries --- in the worst case it could provoke out-of-memory
failures that don't occur now.

Plan B would be for psql to stop using PQerrorMessage and generate the
message report text from the message fields, omitting "at character N".
This would require duplicating a couple dozen lines from inside libpq.
It'd also mean that the report text gets built twice, once inside libpq
and once by psql, which is probably not such a big deal since error
messages ought not be a performance-critical path ... except there's
still that little question of overrunning memory.

Plan C is just to drop the "at character N" string from what
PQerrorMessage generates.  We could make this configurable (maybe via
additional PGVerbosity values), or just not worry about whether it's
important.

Thoughts?

            regards, tom lane

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

Предыдущее
От: Neil Conway
Дата:
Сообщение: Re: Fix syntax errors in contrib uninstall scripts
Следующее
От: Christopher Kings-Lynne
Дата:
Сообщение: Re: Proposed patch for error locations