From: | Marko Tiikkaja <marko(at)joh(dot)to> |
---|---|
To: | Florian Pflug <fgp(at)phlo(dot)org> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Jim Nasby <jim(at)nasby(dot)net> |
Subject: | Re: plpgsql.warn_shadow |
Date: | 2014-01-15 09:08:30 |
Message-ID: | 52D6500E.705@joh.to |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 1/15/14 7:07 AM, Florian Pflug wrote:
> On Jan15, 2014, at 01:34 , Marko Tiikkaja <marko(at)joh(dot)to> wrote:
>> It's me again, trying to find a solution to the most common mistakes I make. This time it's accidental shadowing of variables, especially input variables. I've wasted several hours banging my head against the wall while shouting "HOW CAN THIS VARIABLE ALWAYS BE NULL?". I can't believe I'm the only one. To give you a rough idea on how it works:
>
> I like this, but think that the option should be just called plpgsql.warnings or plpgsql.warn_on and accept a list of warnings to enable.
Hmm. How about:
plpgsql.warnings = 'all' # enable all warnings, defauls to the empty
list, i.e. no warnings
plpgsql.warnings = 'shadow, unused' # enable just "shadow" and
"unused" warnings
plpgsql.warnings_as_errors = on # defaults to off?
This interface is a lot more flexible and should address Jim's concerns
as well.
Regards,
Marko Tiikkaja
From | Date | Subject | |
---|---|---|---|
Next Message | Jan Kara | 2014-01-15 09:35:44 | Re: [Lsf-pc] Linux kernel impact on PostgreSQL performance |
Previous Message | Oleg Bartunov | 2014-01-15 08:46:17 | Re: nested hstore patch - FailedAssertion("!(value->array.nelems == 1) |