diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2014-04-21 13:28:10 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2014-04-21 13:28:10 -0400 |
| commit | 053959397bd1f5e10d16d68bb91b7beffcb673e7 (patch) | |
| tree | e8a4b6d097c8e5362ba425038a92d7e63ef30e87 | |
| parent | 98876f0ff18e617d6366989d40ab74c07580fb0a (diff) | |
| download | postgresql-053959397bd1f5e10d16d68bb91b7beffcb673e7.tar.gz postgresql-053959397bd1f5e10d16d68bb91b7beffcb673e7.zip | |
pg_stat_statements forgot to let previous occupant of hook get control too.
pgss_post_parse_analyze() neglected to pass the call on to any earlier
occupant of the post_parse_analyze_hook. There are no other users of that
hook in contrib/, and most likely none in the wild either, so this is
probably just a latent bug. But it's a bug nonetheless, so back-patch
to 9.2 where this code was introduced.
| -rw-r--r-- | contrib/pg_stat_statements/pg_stat_statements.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/contrib/pg_stat_statements/pg_stat_statements.c b/contrib/pg_stat_statements/pg_stat_statements.c index a6ceaf4f383..8a7e3fe535c 100644 --- a/contrib/pg_stat_statements/pg_stat_statements.c +++ b/contrib/pg_stat_statements/pg_stat_statements.c @@ -614,6 +614,9 @@ pgss_post_parse_analyze(ParseState *pstate, Query *query) { pgssJumbleState jstate; + if (prev_post_parse_analyze_hook) + prev_post_parse_analyze_hook(pstate, query); + /* Assert we didn't do this already */ Assert(query->queryId == 0); |