From: | Dennis Bjorklund <db(at)zigo(dot)dhs(dot)org> |
---|---|
To: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
Cc: | Josh Berkus <josh(at)agliodbs(dot)com>, Alvaro Herrera <alvherre(at)dcc(dot)uchile(dot)cl>, Oliver Jowett <oliver(at)opencloud(dot)com>, Pavel Stehule <stehule(at)kix(dot)fsv(dot)cvut(dot)cz>, Zeugswetter Andreas SB SD <ZeugswetterA(at)spardat(dot)at>, Andreas Pflug <pgadmin(at)pse-consulting(dot)de>, Simon Riggs <simon(at)2ndquadrant(dot)com>, <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Nested Transactions, Abort All |
Date: | 2004-07-10 20:39:48 |
Message-ID: | Pine.LNX.4.44.0407102235100.2838-100000@zigo.dhs.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sat, 10 Jul 2004, Bruce Momjian wrote:
> > > > SAVEPOINT == BEGIN NESTED
> > > > RELEASE SAVEPOINT == COMMIT NESTED
> > > > ROLLBACK TO SAVEPOINT == ROLLBACK NESTED
> > >
> > > Here it should be:
> > >
> > > ROLLBACK TO SAVEPOINT == ROLLBACK NESTED; SAVEPOINT;
> > >
> > > And just to clearify, this is an extension to the spec that we then have
> > > to support for a long time. Adding this now and then replacing it with the
> > > standard syntax is not very sexy. If we add this we should support
> > > it "for ever".
>
> Sorry, I confused folks. I should have corrected this line too:
>
> > > ROLLBACK TO SAVEPOINT == ROLLBACK NESTED; BEGIN NESTED;
Hmm, yes. Correct.
Luckily, we already had: SAVEPOINT == BEGIN NESTED so it all worked out
:-)
> It is not a non-standard behavior. It is only an implementation detail
> used internally that allows nested transactions to implement savepoints.
The non-standard part I was talking about was the savepoints without
names, and that is what we should support for ever if we introduce them.
--
/Dennis Björklund
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2004-07-10 20:45:00 | Re: Nested Transactions, Abort All |
Previous Message | Josh Berkus | 2004-07-10 20:36:58 | Re: Nested Transactions, Abort All |