]> granicus.if.org Git - postgresql/commit
Fix problems with ParamListInfo serialization mechanism.
authorRobert Haas <rhaas@postgresql.org>
Mon, 2 Nov 2015 23:11:29 +0000 (18:11 -0500)
committerRobert Haas <rhaas@postgresql.org>
Mon, 2 Nov 2015 23:11:29 +0000 (18:11 -0500)
commit1efc7e538204646fec0f790b683012ed9bd8a99f
tree713e9f30769f62ac111606a070ce664e524cee18
parentbf25fb2f93c95e946c5e382eb82ad08ae9d95bd9
Fix problems with ParamListInfo serialization mechanism.

Commit d1b7c1ffe72e86932b5395f29e006c3f503bc53d introduced a mechanism
for serializing a ParamListInfo structure to be passed to a parallel
worker.  However, this mechanism failed to handle external expanded
values, as pointed out by Noah Misch.  Repair.

Moreover, plpgsql_param_fetch requires adjustment because the
serialization mechanism needs it to skip evaluating unused parameters
just as we would do when it is called from copyParamList, but params
== estate->paramLI in that case.  To fix, make the bms_is_member test
in that function unconditional.

Finally, have setup_param_list set a new ParamListInfo field,
paramMask, to the parameters actually used in the expression, so that
we don't try to fetch those that are not needed when serializing a
parameter list.  This isn't necessary for correctness, but it makes
the performance of the parallel executor code comparable to what we
do for cases involving cursors.

Design suggestions and extensive review by Noah Misch.  Patch by me.
src/backend/commands/prepare.c
src/backend/executor/functions.c
src/backend/executor/spi.c
src/backend/nodes/params.c
src/backend/tcop/postgres.c
src/backend/utils/adt/datum.c
src/include/nodes/params.h
src/pl/plpgsql/src/pl_exec.c