Re: Issue with ReRaise in PG

From: Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Piyush Newe <piyush(dot)newe(at)enterprisedb(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Issue with ReRaise in PG
Date: 2010-04-23 16:06:50
Message-ID: 4BD1C59A.8010104@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Tom Lane wrote:
> Piyush Newe <piyush(dot)newe(at)enterprisedb(dot)com> writes:
>> Please consider the following test case
>
>> CREATE OR REPLACE FUNCTION raisetest() returns void AS $$
>> BEGIN
>> BEGIN
>> RAISE syntax_error;
>> EXCEPTION
>> WHEN syntax_error THEN
>> BEGIN
>> raise notice 'exception thrown in inner block, reraising';
>> RAISE;
>> EXCEPTION
>> WHEN OTHERS THEN
>> raise notice 'RIGHT - exception caught in innermost
>> block';
>> END;
>> END;
>> EXCEPTION
>> WHEN OTHERS THEN
>> raise notice 'WRONG - exception caught in outer block';
>> END;
>> $$ LANGUAGE plpgsql;
>
>> select raisetest();
>> NOTICE: exception thrown in inner block, reraising
>> NOTICE: WRONG - exception caught in outer block
>
> RAISE without parameters is only allowed inside an exception handler,
> and what it throws is that handler's exception. In this example, it is
> within an exception handler ---- of the outer block. So it's allowed,
> but it re-throws from that handler.
>
> We could possibly make the above case throw an "improperly placed RAISE"
> error instead of doing what it does now, but I don't think there is a
> good argument for having it do what you propose.

It's worth noting that RAISE without parameters was added to mimic the
corresponding RAISE command on Oracle, and on Oracle Piyush's test case
works as he says.

--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2010-04-23 16:09:52 Re: recovery_connections cannot start (was Re: master in standby mode croaks)
Previous Message Terry Brown 2010-04-23 15:58:40 psql: Add setting to make '+' on \d implicit