public inbox for [email protected]  

help / color / mirror / Atom feed

From: Laurenz Albe <[email protected]>
To: Istvan Soos <[email protected]>
Cc: [email protected]
Subject: Re: How to cleanup transaction after statement_timeout aborts a query?
Date: Sun, 08 Sep 2024 15:41:03 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <CALdQGgtteEQYNWhT80gNG3xSsjw+p1xDhFiG-+CUMXR9Gy=wBg@mail.gmail.com>
References: <CALdQGgv_an=JdfkomND7sJcuAS1L5OmbFzhQa9qcW48syTXyKg@mail.gmail.com>
	<[email protected]>
	<CALdQGgtteEQYNWhT80gNG3xSsjw+p1xDhFiG-+CUMXR9Gy=wBg@mail.gmail.com>

On Sun, 2024-09-08 at 15:01 +0200, Istvan Soos wrote:
> On Sun, Sep 8, 2024 at 1:19 PM Laurenz Albe <[email protected]> wrote:
> > ROLLBACK and COMMIT are working: they end the transaction.
> 
> I have this reproduction test, and ROLLBACK does fail:
> https://github.com/isoos/postgresql-dart/pull/363/files#diff-4547e49b04ec8280fb8f4f1ebf695b77f9a2d9a...
> 
> I've checked and nothing else is sent on the protocol, yet, for the
> rollback statement it gets the 25P02 error.

You must be doing something wrong then, because it works:

test=> START TRANSACTION;
START TRANSACTION
test=*> SELECT 1 / 0;
ERROR:  division by zero
test=!> SELECT 42;
ERROR:  current transaction is aborted, commands ignored until end of transaction block
test=!> SELECT 42;
ERROR:  current transaction is aborted, commands ignored until end of transaction block
test=!> ROLLBACK;
ROLLBACK
test=> 

> 
> > To handle the failure of a statement while allowing the transaction to proceed,
> > you can use savepoints.  But be warned: don't even think of setting a savepoint
> > before each statement.  That would affect statement performance severely.
> 
> As the writer of the client library, I don't have the luxury of
> telling users they need to change their way, hence I'm looking for any
> pointer on the protocol level.

I understand.  But there is no way to influence this behavior.

Perhaps the best option for your library is not to try to "mask" the way PostgreSQL
behaves in this case.  It is then up to the users of the library to do the right thing.

Yours,
Laurenz Albe

view thread (2+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: How to cleanup transaction after statement_timeout aborts a query?
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox