Re: Error handling in plperl and pltcl

From: Thomas Hallgren <thhal(at)mailblocks(dot)com>
To: Richard Huxton <dev(at)archonet(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, James William Pye <flaw(at)rhid(dot)com>, Jan Wieck <JanWieck(at)Yahoo(dot)com>, Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Error handling in plperl and pltcl
Date: 2004-12-01 09:12:11
Message-ID: thhal-0biOJAi9RcC4NddpwB/VwwL9d+C9cOt@mailblocks.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Richard Huxton wrote:

> Can I make some counter-proposals?
>
> 1. Wrap each function body/call (same thing here afaict) in a
> sub-transaction. An exception can be caught within that function, and
> all the spi in that function is then rolled back. This is rubbish, but
> at least it's predictable and allows you to write to a log table and
> throw another exception.

This will be even worse since it will impose the subtransaction overhead
on everything, even functions that never do any database access. Perhaps
this approach would be feasible if imposed on volatile functions only,
but then again, the volatility of a function cannot be trusted since we
have no way of defining a "stable but with side effects" type.

> 2. For pl/tcl introduce a pgtry { } catch { } which just starts a
> sub-transaction and does standard try/catch. I don't use TCL, but from
> the little I know this should be straightforward.

If you know how to use special constructs like this, what's wrong with
actually using savepoints verbatim? I.e.

INSERT 1
INSERT 2
SAVEPOINT foo
try {
INSERT 3
INSERT 4
RELEASE foo
}
catch WHATEVER {
ROLLBACK TO foo
INSERT 5
INSERT 6
}

IMHO a very clean, sensible, and easily understood approach that doesn't
clobber the language.

Regards,
Thomas Hallgren

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Richard Huxton 2004-12-01 09:27:07 Re: Error handling in plperl and pltcl
Previous Message Richard Huxton 2004-12-01 08:57:32 Re: createdb failed