On Thu, Apr 27, 2017 at 11:05 PM, Huong Dangminh <huo-dangm...@ys.jp.nec.com> wrote: >> >>> I would refrain from doing that, having some parameters listed in the >> >>> tests makes the intention behind those perl routines clear. >> > >> > Hmm, you've got a point. But when we changed the default values >> > related to replication we dropped some explicitly settings from the >> > regression test code. >> >> Looking at the patch. This is fine: >> - # Change a setting and restart >> - $node->append_conf('postgresql.conf', 'hot_standby = on'); >> - $node->restart(); >> >> But not that: >> print $conf "wal_log_hints = on\n"; >> - print $conf "hot_standby = on\n"; >> print $conf "max_connections = 10\n"; >> >> This is a minor point though.
After some thoughts I agree to remain it in the perl code. > > Thanks, I attached the update patch. So it looks good to me. Regards, -- Masahiko Sawada NIPPON TELEGRAPH AND TELEPHONE CORPORATION NTT Open Source Software Center -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers