diff options
| author | Kevin Grittner | 2017-04-01 20:21:05 +0000 |
|---|---|---|
| committer | Kevin Grittner | 2017-04-01 20:21:05 +0000 |
| commit | 41bd155dd656e7f17c02855be7aff234843347cd (patch) | |
| tree | 2b5181cbab8739e2772fa0eb3d7f72d6df0a795f /src/backend/tcop/postgres.c | |
| parent | c655899ba9ae2a0d24e99c797167c33e0cfa0820 (diff) | |
Fix two undocumented parameters to functions from ENR patch.
On ProcessUtility document the parameter, to match others.
On CreateCachedPlan drop the queryEnv parameter. It was not
referenced within the function, and had been added on the
assumption that with some unknown future usage of QueryEnvironment
it might be useful to do something there. We have avoided other
"just in case" implementation of unused paramters, so drop it here.
Per gripe from Tom Lane
Diffstat (limited to 'src/backend/tcop/postgres.c')
| -rw-r--r-- | src/backend/tcop/postgres.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c index 139c4c0f68..a2282058c0 100644 --- a/src/backend/tcop/postgres.c +++ b/src/backend/tcop/postgres.c @@ -1318,8 +1318,7 @@ exec_parse_message(const char *query_string, /* string to execute */ * Create the CachedPlanSource before we do parse analysis, since it * needs to see the unmodified raw parse tree. */ - psrc = CreateCachedPlan(raw_parse_tree, query_string, commandTag, - NULL); + psrc = CreateCachedPlan(raw_parse_tree, query_string, commandTag); /* * Set up a snapshot if parse analysis will need one. @@ -1371,8 +1370,7 @@ exec_parse_message(const char *query_string, /* string to execute */ /* Empty input string. This is legal. */ raw_parse_tree = NULL; commandTag = NULL; - psrc = CreateCachedPlan(raw_parse_tree, query_string, commandTag, - NULL); + psrc = CreateCachedPlan(raw_parse_tree, query_string, commandTag); querytree_list = NIL; } |