diff options
| author | Kevin Grittner <kgrittn@postgresql.org> | 2017-04-01 15:21:05 -0500 |
|---|---|---|
| committer | Kevin Grittner <kgrittn@postgresql.org> | 2017-04-01 15:21:05 -0500 |
| commit | 41bd155dd656e7f17c02855be7aff234843347cd (patch) | |
| tree | 2b5181cbab8739e2772fa0eb3d7f72d6df0a795f /src/backend/tcop | |
| parent | c655899ba9ae2a0d24e99c797167c33e0cfa0820 (diff) | |
| download | postgresql-41bd155dd656e7f17c02855be7aff234843347cd.tar.gz postgresql-41bd155dd656e7f17c02855be7aff234843347cd.zip | |
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')
| -rw-r--r-- | src/backend/tcop/postgres.c | 6 | ||||
| -rw-r--r-- | src/backend/tcop/utility.c | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c index 139c4c0f68c..a2282058c0d 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; } diff --git a/src/backend/tcop/utility.c b/src/backend/tcop/utility.c index b610c8e7cee..c0fd0f967f6 100644 --- a/src/backend/tcop/utility.c +++ b/src/backend/tcop/utility.c @@ -308,6 +308,8 @@ CheckRestrictedOperation(const char *cmdname) * context: identifies source of statement (toplevel client command, * non-toplevel client command, subcommand of a larger utility command) * params: parameters to use during execution + * queryEnv: environment for parse through execution (e.g., ephemeral named + * tables like trigger transition tables). May be NULL. * dest: where to send results * completionTag: points to a buffer of size COMPLETION_TAG_BUFSIZE * in which to store a command completion status string. |