From: | Petr Jelinek <pjmodos(at)pjmodos(dot)net> |
---|---|
To: | Dimitri Fontaine <dfontaine(at)hi-media(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
Subject: | Re: Anonymous code blocks |
Date: | 2009-09-22 13:01:58 |
Message-ID: | 4AB8CAC6.2070705@pjmodos.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Dimitri Fontaine napsal(a):
> Hi,
>
> Dimitri Fontaine <dfontaine(at)hi-media(dot)com> writes:
>
>> Patch applies cleanly and build cleanly too, basic examples are working
>> fine.
>>
>
> I've been reading through the code and am going to mark it as ready for
> commiter, as only remarks I have are probably because I do not know
> enough about PostgreSQL internals, and the one I missed are in the same
> category.
>
> The patch is easy to read and all it does looks straightforward, even
> for me :)
>
> Here we go:
>
> *** a/src/backend/tcop/utility.c
> --- b/src/backend/tcop/utility.c
> ...
> *************** UtilityReturnsTuples(Node *parsetree)
> *** 1147,1155 ****
> ...
> - case T_ExplainStmt:
> - return true;
> -
>
> Is this not a oversight in the final patch?
>
It is. I attached patch which does not have this part.
>
> + /* This is short-lived, so needn't allocate in function's cxt */
> + plpgsql_Datums = palloc(sizeof(PLpgSQL_datum *) * datums_alloc);
> ...
> + compile_tmp_cxt = MemoryContextSwitchTo(func_cxt);
>
> I wonder why not having the datums into the func_cxt too.
>
Actually I think we might not need that function memory context for
anonymous code blocks at all since we don't cache compiled functions.
But I am not sure so I basically copied it from standard function
compiler to be on safe side. I am sure commiter will comment on this :)
--
Regards
Petr Jelinek (PJMODOS)
Attachment | Content-Type | Size |
---|---|---|
inlinepl-2009-09-22.diff.gz | application/x-tar | 9.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2009-09-22 13:29:13 | Re: Hot Standby 0.2.1 |
Previous Message | Euler Taveira de Oliveira | 2009-09-22 12:51:57 | Re: numeric_to_number() function skipping some digits |