Bug #120488 for Database-Migrator-Pg: Should set ON_ERROR

Fri Mar 03 10:44:31 2017 OSCHWALD [...] cpan.org - Ticket created

Subject:

Should set ON_ERROR_STOP

Currently, psql continues processing the script if there is an error. Setting ON_ERROR_STOP seems like a better default that is less likely to cause an unintended state if part of a migration script fails. Greg

Tue Mar 07 15:54:38 2017 DROLSKY [...] cpan.org - Correspondence added

On 2017-03-03 09:44:31, OSCHWALD wrote: Show quoted text

> Currently, psql continues processing the script if there is an error. > Setting ON_ERROR_STOP seems like a better default that is less likely > to cause an unintended state if part of a migration script fails.

I agree. Patches are welcome, or I can give you comaint on this. I don't know that I'll be using this at work any time soon, and I know you are.

Tue Mar 07 15:54:38 2017 The RT System itself - Status changed from 'new' to 'open'

Tue Mar 07 23:11:55 2017 oschwald [...] gmail.com - Correspondence added

CC:	OSCHWALD [...] cpan.org
Subject:	Re: [rt.cpan.org #120488] Should set ON_ERROR_STOP
Date:	Tue, 7 Mar 2017 20:11:37 -0800
To:	Dave Rolsky via RT <bug-Database-Migrator-Pg [...] rt.cpan.org>
From:	"Gregory J. Oschwald" <oschwald [...] gmail.com>

Show quoted text

> I agree. Patches are welcome, or I can give you comaint on this. I don't > know that I'll be using this at work any time soon, and I know you are.

I did this in: https://github.com/houseabsolute/Database-Migrator-Pg/pull/1 I can also take comaint if you prefer doing it that way. Thanks, Greg

Fri May 12 09:56:29 2017 OSCHWALD [...] cpan.org - Status changed from 'open' to 'resolved'

Fri May 12 09:56:29 2017 OSCHWALD [...] cpan.org - Fixed in 0.06 added

Bug #120488 for Database-Migrator-Pg: Should set ON_ERROR_STOP