[pgpool-hackers: 3493] Re: [Proposal] Check if socket file exists at startup and remove them if PID file doesn't exist.
Tatsuo Ishii
ishii at sraoss.co.jp
Wed Jan 22 12:01:02 JST 2020
>> The patch looks good to me. Also I think we should back patch this to
>> all supported branches.
>
> Thank you for reviewing my patch.
> Yes. I agree to back patch it to all supported branches.
You are welcome.
BTW, although not directly connected this patch, I found a bug with
Pgpool-II start up process while checking the patch. Around line 279
of main/main.c:
if (pid > 0)
{
if (kill(pid, 0) == 0)
{
ereport(FATAL,
(errmsg("pid file found. is another pgpool(%d) is running?\n", pid)));
}
else
ereport(NOTICE,
(errmsg("pid file found but it seems bogus. Trying to start pgpool anyway...\n")));
}
Since elog system is not yet initialized at this point, those messages
are effectively ignored. Probably we should change them to standard
fprintf(stderr...) or something like that.
Best regards,
--
Tatsuo Ishii
SRA OSS, Inc. Japan
English: http://www.sraoss.co.jp/index_en.php
Japanese:http://www.sraoss.co.jp
More information about the pgpool-hackers
mailing list