Re: Trivial heap_finish_speculative() error message inaccuracy

Поиск
Список
Период
Сортировка
От Andres Freund
Тема Re: Trivial heap_finish_speculative() error message inaccuracy
Дата
Msg-id 20151118225030.GA5217@alap3.anarazel.de
обсуждение исходный текст
Ответ на Re: Trivial heap_finish_speculative() error message inaccuracy  (Peter Geoghegan <pg@heroku.com>)
Ответы Re: Trivial heap_finish_speculative() error message inaccuracy
Список pgsql-hackers
On 2015-11-03 19:14:44 -0800, Peter Geoghegan wrote:
> On Tue, Nov 3, 2015 at 7:10 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> > This seems like a fine teaching moment in which to point out our
> > longstanding error message style guideline that says not to put
> > names of C functions into error messages in the first place.
> 
> I don't ordinarily do that, of course, but I thought it was important
> to be consistent with other such elog() calls within heapam.c.

Internal, super unlikely, elog()s don't seem to need much consistency
among themselves.

> I think that there at least 10 that look like this.

Let's rather rip those function names out. Unless somebody protests I'm
going to do so in 9.5/master.

Andres



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

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Bug in numeric multiplication
Следующее
От: Peter Geoghegan
Дата:
Сообщение: Re: Using quicksort for every external sort run