Re: PATCH: Unlogged tables re-initialization tests - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: PATCH: Unlogged tables re-initialization tests
Date
Msg-id 65c7dbb0-2e6f-f97e-69d2-20d4fc781127@2ndquadrant.com
Whole thread Raw
In response to Re: PATCH: Unlogged tables re-initialization tests  (Michael Paquier <michael@paquier.xyz>)
Responses Re: PATCH: Unlogged tables re-initialization tests  (David Steele <david@pgmasters.net>)
List pgsql-hackers
On 3/11/18 05:11, Michael Paquier wrote:
> On Fri, Mar 09, 2018 at 05:23:48PM -0500, Peter Eisentraut wrote:
>> This seems like a useful test.
>>
>> On 3/5/18 12:35, David Steele wrote:
>>> +mkdir($tablespaceDir)
>>> +    or die "unable to mkdir \"$tablespaceDir\"";
>>
>> Use BAIL_OUT instead of die in tests.
> 
> Would it be better to make this practice more uniform?  From the code of
> the tests:
> $ git grep die -- */t/*.pl | wc -l
> 50

Yes, or maybe there is a way to "catch" die and turn it into BAIL_OUT?

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


pgsql-hackers by date:

Previous
From: Nikita Glukhov
Date:
Subject: Re: Jsonb transform for pl/python
Next
From: David Steele
Date:
Subject: Re: PATCH: Unlogged tables re-initialization tests