Re: [PATCH] SQL function to report log message - Mailing list pgsql-hackers

From Jim Nasby
Subject Re: [PATCH] SQL function to report log message
Date
Msg-id 56227A74.80304@BlueTreble.com
Whole thread Raw
In response to Re: [PATCH] SQL function to report log message  (Pavel Stehule <pavel.stehule@gmail.com>)
Responses Re: [PATCH] SQL function to report log message
List pgsql-hackers
On 10/15/15 11:51 PM, Pavel Stehule wrote:
> I don't think so ignoring NULL in RAISE statement is good idea (it is
> not safe). We can replace NULL by some string (like "NULL") by default.
> I am thinking about other possibilities.

What I was trying to say is that if the argument to a USING option is 
NULL then RAISE should skip over it, as if it hadn't been applied at 
all. Similar to how the code currently tests for \0.

> 1. some RAISE statement flag - but there was strong disagreement when I
> did it last time
> 2. some plpgsql GUC variables like plpgsq.raise_ignore_null
> 3. accept a function from this patch
>
> Now, I am thinking so @3 is good option. It can be really useful as last
> rescue for other PL without possibility to raise rich PostgreSQL
> exception - currently PLPythonu, partially PLPerl (where are more
> issues), probably in others.

I agree, assuming the patch exposes all the stuff you can do with USING 
in plpgsql.
-- 
Jim Nasby, Data Architect, Blue Treble Consulting, Austin TX
Experts in Analytics, Data Architecture and PostgreSQL
Data in Trouble? Get it in Treble! http://BlueTreble.com



pgsql-hackers by date:

Previous
From: Jim Nasby
Date:
Subject: Re: Improve the concurency of vacuum full table and select statement on the same relation
Next
From: Pavel Stehule
Date:
Subject: Re: [PATCH] SQL function to report log message