Skip site navigation (1) Skip section navigation (2)

Re: nested xacts and phantom Xids

From: Alvaro Herrera <alvherre(at)dcc(dot)uchile(dot)cl>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: nested xacts and phantom Xids
Date: 2004-06-24 23:53:27
Message-ID: 20040624235327.GD8190@dcc.uchile.cl (view raw or flat)
Thread:
Lists: pgsql-hackerspgsql-patches
On Sun, Jun 20, 2004 at 08:49:22PM -0400, Tom Lane wrote:

Regarding GUC, a WIP report:

> Given patches for inval.c and guc.c, I would say that the patch is
> functionally close enough to done that we could commit to including
> it in 7.5 --- the other stuff could be wrapped up post-feature-freeze.

I figured I could save the values whenever they are going to change, and
restore them if the subtransaction aborts.  This seems to work fine
(lightly tested).

I still have to figure out how to handle allocation for string vars, but
I thought I'd post the patch for others to see.  Please let me know if
it's too ugly.  (This patch misses the pieces in xact.c and xact.h but
I'm sure the concept is clear.)

I'll post a full patch once the missing deferred trigger stuff works.
With the patches I posted to inval.c I think this fulfills the
requirements, barring the performance issues raised.

Comments?

-- 
Alvaro Herrera (<alvherre[a]dcc.uchile.cl>)
"No single strategy is always right (Unless the boss says so)"
(Larry Wall)

In response to

Responses

pgsql-hackers by date

Next:From: Greg Sabino MullaneDate: 2004-06-25 02:00:12
Subject: Re: [Re] Re: PREPARE and transactions
Previous:From: Tom LaneDate: 2004-06-24 21:39:15
Subject: Shouldn't we be using openlog's LOG_NOWAIT option?

pgsql-patches by date

Next:From: Tom LaneDate: 2004-06-25 02:57:33
Subject: Re: Add argument name to CREATE FUNCTION documentation
Previous:From: Dennis BjorklundDate: 2004-06-24 22:05:24
Subject: Add argument name to CREATE FUNCTION documentation

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group