[pgpool-hackers: 4168] Re: Allow pgpool_setup to test sample scripts and config files included in RPMs

Tatsuo Ishii ishii at sraoss.co.jp
Wed Jun 8 22:02:20 JST 2022


Hi Peng,

> Thank you for testing my patch.
> I have fixed this timeout error. Patch is attached.
> 
> Because I have changed the backend_hostname to "lcoalhost",
> I need to change $PGSOCKET_DIR to "localhost" in test 055.
> 
>   grep "Failover done. shutdown host $PGSOCKET_DIR(11003)" log/pgpool.log > /dev/null 2>&1

After applying the new patch all the regression tests passed. Great!

I would suggest to update the usage message of pgpool_setup so that
the new "-c" option is shown. Also it would be nice if you update the
pgpool_setup manual to reflect your changes.

Best reagards,
--
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