2012/1/26 Abhijit Menon-Sen <ams(at)toroid(dot)org>:
> At 2011-11-24 17:44:16 +0100, pavel(dot)stehule(at)gmail(dot)com wrote:
>> patch is relative long, but almost all are changes in regress tests.
>> Changes in plpgsql are 5 lines
> The change looks good in principle. The patch applies to HEAD with a bit
> of fuzz and builds fine… but it fails tests, because it's incomplete.
> Pavel, your patch doesn't contain any changes to pl_exec.c. Did you just
> forget to submit them? Anyway, some errcontext() calls need to be taught
> to print ->fn_signature rather than ->fn_name. I made those changes, and
> found some more failing tests.
It was my mistake - using fn_signature for runtime errors is good idea
> Updated patch attached. Ready for committer.
I found a small issue - there was uninitialized fn_signature for
online blocks so I append line
function->fn_signature = pstrdup(func_name); to
plpgsql_compile_inline(char *proc_source) function
modified patch is in attachment
> -- ams
In response to
pgsql-hackers by date
|Next:||From: Fred&Dani&Pandora||Date: 2012-01-27 13:58:59|
|Subject: Re: Multithread Query Planner|
|Previous:||From: Robert Haas||Date: 2012-01-27 13:38:57|
|Subject: Re: Group commit, revised|