On 1 March 2016 at 21:05, Craig Ringer <cr...@2ndquadrant.com> wrote:

> On 1 March 2016 at 20:45, Michael Paquier <michael.paqu...@gmail.com>
> wrote:
>> On Tue, Mar 1, 2016 at 5:13 PM, Kyotaro HORIGUCHI
>> <horiguchi.kyot...@lab.ntt.co.jp> wrote:
>> > 0001-Change-behavior-...
>> >
>> >   Changes of PostmasterNode.pm and TestLib.pm to add some
>> >   features and change a behavior.
>> +   # Preserve temporary directory for this test if failure
>> +   $File::Temp::KEEP_ALL = 1 unless Test::More->builder->is_passing;
>> +1. Having the data folders being removed even in case of a failure is
>> really annoying.
> I agree on all points re your review. Keeping tempdirs is really needed,
> the tempdir name change is good, the  the rest I'm not keen on.
> I've addressed the need to get stderr from psql in a patch I'll submit
> separately, which provides a thinner wrapper around IPC::Run for more
> complex needs, then uses that for the existing 'psql' function. It also
> provides a 'psql_check' that dies on error.
> I'll incorporate the wanted changes into that patch.

OK, done.


Michael, I'd value your thoughts on the patches. I think the psql one is
important, I found the existing 'psql' function too limiting and I think
defaulting to ignoring errors is sub-optimal.

 Craig Ringer                   http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services

Reply via email to