Re: PL/pgSQL, RAISE and error context - Mailing list pgsql-hackers

From Marko Tiikkaja
Subject Re: PL/pgSQL, RAISE and error context
Date
Msg-id 5233D0E7.6050508@joh.to
Whole thread Raw
In response to Re: PL/pgSQL, RAISE and error context  (Marko Tiikkaja <marko@joh.to>)
Responses Re: PL/pgSQL, RAISE and error context
Re: PL/pgSQL, RAISE and error context
List pgsql-hackers
On 23/08/2013 10:36, I wrote:
> On 8/23/13 8:38 AM, Pavel Stehule wrote:
>> do you prepare patch ?
>
> I should have the time to produce one for the September commitfest, but
> if you (or anyone else) want to work on this, I won't object.
>
> My opinion at this very moment is that we should leave the the DEFAULT
> verbosity alone and add a new one (call it COMPACT or such) with the
> suppressed context for non-ERRORs.

Well, turns out there isn't really any way to preserve complete
backwards compatibility if we want to do this change.

The attached patch (based on Pavel's patch) changes the default to be
slightly more verbose (the CONTEXT lines which were previously omitted
will be visible), but adds a new PGVerbosity called COMPACT which
suppresses CONTEXT in non-error messages.  Now DEFAULT will be more
useful when debugging PL/PgSQL, and people who are annoyed by the new
behaviour can use the COMPACT mode.

Any thoughts?



Regards,
Marko Tiikkaja


Attachment

pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: [PATCH] Add use of asprintf()
Next
From: Marko Tiikkaja
Date:
Subject: Proposal: PL/PgSQL strict_mode