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

Copy toplevel queryId to citus' master statement. 

multi_ExecutorStart() replaces the original planned statement with the
master select statement. As that hasn't gone through the parse analysis
hooks, it'll not have a associated queryId.  This prevents extensions
pg_stat_statements to show useful data associated with the query.
pull/383/head
Andres Freund 2016-03-14 17:27:52 -07:00
parent 692563930f
commit b60f5da774
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/backend/distributed/executor/multi_executor.c
View File

@ -153,8 +153,14 @@ multi_ExecutorStart(QueryDesc *queryDesc, int eflags)
* the later hooks, so we set a flag marking this as a distributed * the later hooks, so we set a flag marking this as a distributed
* statement running on the master. That e.g. allows us to drop the * statement running on the master. That e.g. allows us to drop the
* temp table later. * temp table later.
*
* We copy the original statement's queryId, to allow
* pg_stat_statements and similar extension to associate the
* statement with the toplevel statement.
*/ */
masterSelectPlan->queryId = queryDesc->plannedstmt->queryId;
queryDesc->plannedstmt = masterSelectPlan; queryDesc->plannedstmt = masterSelectPlan;
eflags |= EXEC_FLAG_CITUS_MASTER_SELECT; eflags |= EXEC_FLAG_CITUS_MASTER_SELECT;
} }
} }