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.
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 |
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) |