Re: [PATCH] V3: Idle in transaction cancellation - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: [PATCH] V3: Idle in transaction cancellation
Date
Msg-id 1288734369-sup-3437@alvh.no-ip.org
Whole thread Raw
In response to Re: [PATCH] V3: Idle in transaction cancellation  (Andres Freund <andres@anarazel.de>)
List pgsql-hackers
Excerpts from Andres Freund's message of mar nov 02 18:36:19 -0300 2010:
> On Tuesday 02 November 2010 18:33:15 Kevin Grittner wrote:
> > Andres Freund <andres@anarazel.de> wrote:
> > > * You wont see an error if the next command after the IDLE in
> > > transaction is a COMMIT/ROLLBACK. I don*t see any sensible way
> > > around that.
> > Well, on a ROLLBACK I'm not sure it's a problem.  On a COMMIT,
> > couldn't you call a function to check for it in CommitTransaction
> > and PrepareTransaction?
> Sure, throwing an error somewhere wouldnt be that hard. But at the moment a 
> COMMIT is always successfull (and just reporting a ROLLBACK in a failed txn). 
> Doesn't seem to be something to changed lightly.

If the user calls COMMIT, it calls EndTransactionBlock, which ends up
calling AbortTransaction.  Can't you do it at that point?  (Says he who
hasn't looked at the patch at all)

-- 
Álvaro Herrera <alvherre@commandprompt.com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Hash support for arrays
Next
From: "Kevin Grittner"
Date:
Subject: Re: [PATCH] V3: Idle in transaction cancellation