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:49:28
Message-ID: CAKFQuwbPxVT+bKoGdT1KCmJUBSCGgQw+Epv7AbndcrDwWpP_Zw@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: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.

David J.


In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message David Johnston 2014-12-29 16:52:53 Re: Rollback on include error in psql
Previous Message Nikita Volkov 2014-12-29 16:47:09 Re: BUG #12330: ACID is broken for unique constraints

Browse pgsql-general by date

  From Date Subject
Next Message Adrian Klaver 2014-12-29 16:51:14 ON_ERROR_ROLLBACK
Previous Message Adrian Klaver 2014-12-29 16:41:54 Re: localtime(0)