diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2013-04-28 00:18:45 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2013-04-28 00:18:45 -0400 |
commit | f8db76e875099e5e49f5cd729a673e84c0b0471b (patch) | |
tree | 162e43ab4a10fbeef78201e9adc87c433fe9d197 /src/backend/executor/spi.c | |
parent | 5525e6c40bbda351a19b48317eba0f79aa32e447 (diff) | |
download | postgresql-f8db76e875099e5e49f5cd729a673e84c0b0471b.tar.gz postgresql-f8db76e875099e5e49f5cd729a673e84c0b0471b.zip |
Editorialize a bit on new ProcessUtility() API.
Choose a saner ordering of parameters (adding a new input param after
the output params seemed a bit random), update the function's header
comment to match reality (cmon folks, is this really that hard?),
get rid of useless and sloppily-defined distinction between
PROCESS_UTILITY_SUBCOMMAND and PROCESS_UTILITY_GENERATED.
Diffstat (limited to 'src/backend/executor/spi.c')
-rw-r--r-- | src/backend/executor/spi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/executor/spi.c b/src/backend/executor/spi.c index de8d59a8cdc..ca0d05d2cc0 100644 --- a/src/backend/executor/spi.c +++ b/src/backend/executor/spi.c @@ -2093,10 +2093,10 @@ _SPI_execute_plan(SPIPlanPtr plan, ParamListInfo paramLI, ProcessUtility(stmt, plansource->query_string, + PROCESS_UTILITY_QUERY, paramLI, dest, - completionTag, - PROCESS_UTILITY_QUERY); + completionTag); /* Update "processed" if stmt returned tuples */ if (_SPI_current->tuptable) |