From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: proposal: better support for debugging of overloaded functions |
Date: | 2011-11-20 11:16:56 |
Message-ID: | CAFj8pRAOQOF59WUgw72RRVHr9VUiEn+kPA9mXfyOOaO1bJiFpg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
2011/11/18 Robert Haas <robertmhaas(at)gmail(dot)com>:
> On Fri, Nov 18, 2011 at 6:24 AM, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> wrote:
>> CONTEXT: PL/pgSQL function "assign_rslts" line 50 at assignment (oid: 65903)
>>
>> \sf+ 65903
>
> I'm pretty unenthused by the idea of making OIDs more user-visible
> than they already are. If the message is ambiguous, we should include
> argument types and (if not the object that would be visible under the
> current search_path) a schema qualification. Spitting out a five (or
> six or seven or eight) digit number doesn't seem like a usability
> improvement.
Is possible to add GUC variable plpgsql.log_function_signature (maybe
just log_function_signature (for all PL))? I am not sure about GUC
name.
When this variable is true, then CONTEXT line will contain a qualified
function's signature instead function name
I don't would to check if function name is ambiguous or not after
exception is raised. There is a problem with access to system tables
and then exception handling can be slower. Using a qualified name is
necessary, because psql meta statements are not "smart" - they are
based on search_path and fact, so name is not ambiguous doesn't help
there.
Regards
Pavel Stehule
p.s. Other issue is missing CONTEXT line for RAISE EXCEPTION
>
> --
> Robert Haas
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
From | Date | Subject | |
---|---|---|---|
Next Message | Gianni Ciolli | 2011-11-20 12:58:26 | Re: [PATCH] Support for foreign keys with arrays |
Previous Message | Gabriele Bartolini | 2011-11-20 09:36:15 | Re: [PATCH] Support for foreign keys with arrays |