Re: Rollback on include error in psql

From: David Johnston <david(dot)g(dot)johnston(at)gmail(dot)com>
To: Adrian Klaver <adrian(dot)klaver(at)aklaver(dot)com>
Cc: "pgsql-general(at)postgresql(dot)org" <pgsql-general(at)postgresql(dot)org>
Subject: Re: Rollback on include error in psql
Date: 2014-12-29 16:52:53
Message-ID: CAKFQuwZmNdxL9s1CLjm9-akPyN2u8A-QCA0nbAnYn91473ZnnQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-general

On Mon, Dec 29, 2014 at 9:49 AM, David Johnston <david(dot)g(dot)johnston(at)gmail(dot)com>
wrote:

> On Mon, Dec 29, 2014 at 9:39 AM, Adrian Klaver <adrian(dot)klaver(at)aklaver(dot)com>
> wrote:
>
>> On 12/29/2014 07:59 AM, David Johnston wrote:
>>
>>>
>>> Anyway, the third undocumented bug is that --single-transactions gets to
>>> send its COMMIT even if ON_ERROR_STOP​
>>> ​takes hold before the end of the script. I imagined it such that only
>>> if every statement in the "-f <script>" was called would the COMMIT be
>>> issued - thus the error_stop would supercede and leave the session
>>> uncommitted and by default rolledback.
>>>
>>
>> Not seeing the bug. --single-transaction wraps the entire script in
>> BEGIN/COMMIT, ON_ERROR_STOP stops 'processing' the command, nothing in
>> there about stopping transaction or rollback. So the failed \i stops the
>> script from processing anything after that and the session goes directly to
>> the COMMIT. If you want to deal with transactions there is
>> ON_ERROR_ROLLBACK. Though I did find something interesting about that,
>> which will subject of another post.
>>
>>
> ​Then --single-transaction has nothing to do with the script file at-all.
> It should be documented as issuing a BEGIN at session connect and a COMMIT
> just before session disconnect - regardless of whether the named script
> executes to completion, which can happen if it is combined with
> ON_ERROR_STOP.
>
>
>
​FWIW​

​The way this is written currently I am imagining something like this
happens:

cat "BEGIN;" filename "COMMIT;" > script_to_execute​

\i script_to_execute

David J.

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Kevin Grittner 2014-12-29 16:53:24 Re: BUG #12330: ACID is broken for unique constraints
Previous Message David Johnston 2014-12-29 16:49:28 Re: Rollback on include error in psql

Browse pgsql-general by date

  From Date Subject
Next Message Andy Colson 2014-12-29 17:00:00 Re: Improving performance of merging data between tables
Previous Message Adrian Klaver 2014-12-29 16:51:14 ON_ERROR_ROLLBACK