Re: [HACKERS] Anonymous code blocks

[Petr Jelinek]

Dimitri Fontaine napsal(a):

Hi,

Dimitri Fontaine <dfonta...@hi-media.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

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?

category. -

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.

--
Regards
Petr Jelinek (PJMODOS)

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 :)

inlinepl-2009-09-22.diff.gz
Description: Unix tar archive

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers