]> granicus.if.org Git - postgresql/commit
It seems that SPI_prepare() doesn't work well in some cases.
authorBruce Momjian <bruce@momjian.us>
Thu, 21 Jan 1999 22:55:41 +0000 (22:55 +0000)
committerBruce Momjian <bruce@momjian.us>
Thu, 21 Jan 1999 22:55:41 +0000 (22:55 +0000)
commit03155dc7649d85ec1ea6fecdaf6e9f8094fa6830
treee975bf625a02e69fb2be6b97bb87acf39b888d54
parent289a826dab09844cb7df41828d60ac7aef35125c
It seems that SPI_prepare() doesn't work well in some cases.

Pawel Pierscionek [pawel@astercity.net] reported about the
following case 1([SQL] drop table in pgsql).
Michael Contzen [mcontzen@dohle.com] reported about the
following case 2(PL/PGSQL bug using aggregates).
You can find it from pgsql-hackers archive.

1. PL/pgSQL can't execute UTILITY commands.
   SPI_prepare() doesn't copy(save) the utilityStmt member of
   Query type nodes,because copyObject() is not implemented
   for nodes of (Create/Destroy etc)Stmt type.

2. Aggregates in PL/pgSQL cause wrong results.

...
   It's a list including Aggreg type nodes which exist in
   TargetList(i.e Aggreg type nodes are common to aggs
   member list and TargetList).
   AFAIC the common pointer is not copied to the same
   pointer by copyObject() function.
   In my patch I reconstruct aggs member node from
   new(copied) Agg type node.
   Is it proper to use set_agg_tlist_references() function to
   reconstruct aggs member node for Agg type nodes ?

Thanks.

Hiroshi Inoue
Inoue@tpf.co.jp
src/backend/nodes/copyfuncs.c