Re: debug log in pg_archivecleanup

Поиск
Список
Период
Сортировка
Искать
От
Simon Riggs
Тема
Re: debug log in pg_archivecleanup
Дата
Msg-id
1276851076.23257.81482.camel@ebony
Ответ на
Re: debug log in pg_archivecleanup (Tom Lane)
Список
Дерево обсуждения
debug log in pg_archivecleanup Fujii Masao <masao.fujii@gmail.com>
Re: debug log in pg_archivecleanup Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp>
Re: debug log in pg_archivecleanup Fujii Masao <masao.fujii@gmail.com>
Re: debug log in pg_archivecleanup Tom Lane <tgl@sss.pgh.pa.us>
Re: debug log in pg_archivecleanup Simon Riggs <simon@2ndQuadrant.com> 
On Thu, 2010-06-17 at 13:33 -0400, Tom Lane wrote:
> Fujii Masao  writes:
> > On Wed, Jun 16, 2010 at 12:24 PM, Takahiro Itagaki
> >  wrote:
> >> This is because pg_archivecleanup puts the line break "\n" in the head of
> >> debug message. Why should we do so?
> 
> > Yes. What about the attached patch?
> 
> Applied along with a bit of further editorialization.
> 
> >> Note that we don't need a line break at Line 130
> >> because strerror() fills the last %s.
> >> L.130: fprintf(stderr, "\n%s: ERROR failed to remove \"%s\": %s",
> 
> > Right.
> 
> Huh?  strerror() doesn't include a newline.

Thanks.

-- Simon Riggs           www.2ndQuadrant.comPostgreSQL Development, 24x7 Support, Training and Services
В списке pgsql-hackers по дате отправления
Предыдущее
От: Simon Riggs
Дата:
Следующее
От: Thom Brown
Дата:
FAQ