external
/
citus
mirror of https://github.com/citusdata/citus.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Skip over unreferenced parameters when router executing prepared statement. 

When an unreferenced prepared statement parameter does not explicitly
have a type assigned, we cannot deserialize it, to send to the remote
side.  That commonly happens inside plpgsql functions, where local
variables are passed in as unused prepared statement parameters.
pull/718/head
Andres Freund 2016-08-05 14:07:42 -07:00
parent 27171c230c
commit 7f6084c14f
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/backend/distributed/executor/multi_router_executor.c
View File

@ -821,7 +821,12 @@ ExtractParametersFromParamListInfo(ParamListInfo paramListInfo, Oid **parameterT
(*parameterTypes)[parameterIndex] = parameterData->ptype; (*parameterTypes)[parameterIndex] = parameterData->ptype;
} }
if (parameterData->isnull) /*
* If the parameter is NULL, or is not referenced / used (ptype == 0
* would otherwise have errored out inside standard_planner()),
* don't pass a value to the remote side.
*/
if (parameterData->isnull || parameterData->ptype == 0)
{ {
(*parameterValues)[parameterIndex] = NULL; (*parameterValues)[parameterIndex] = NULL;
continue; continue;