Re: pgsql: Fix for early log messages during postmaster startup getting lost

Поиск
Список
Период
Сортировка
От Neil Conway
Тема Re: pgsql: Fix for early log messages during postmaster startup getting lost
Дата
Msg-id 1171251947.5454.200.camel@localhost.localdomain
обсуждение исходный текст
Ответ на pgsql: Fix for early log messages during postmaster startup getting lost  (mha@postgresql.org (Magnus Hagander))
Ответы Re: pgsql: Fix for early log messages during postmaster startup getting lost  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-committers
On Sun, 2007-02-11 at 07:59 -0400, Magnus Hagander wrote:
> Fix for early log messages during postmaster startup getting lost when
> running as a service on Win32.

FYI, it is considered good practise to commit a patch at approximately
(or exactly) the same time on all branches, so tools like cvs2cl will be
more likely to collect the changes together.

>     pgsql/src/backend/postmaster:
>         postmaster.c (r1.519 -> r1.520)
>         (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/postmaster/postmaster.c.diff?r1=1.519&r2=1.520)

Comments like that are fragile (elog.c could change, for example), and
basically content-free anyway, IMHO. If you need to make SysLoggerPID
part of postmaster.c's external API, why not just do that, remove the
comment, and add the extern declaration to postmaster.h?

-Neil



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

Предыдущее
От: mha@postgresql.org (User Mha)
Дата:
Сообщение: pgsql: Fix pg_standby to build on msvc.
Следующее
От: Tom Lane
Дата:
Сообщение: Re: pgsql: Fix for early log messages during postmaster startup getting lost