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. https://commitfest.postgresql.org/9/569/# 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