postgres.git / summary / log / commit / refs
commit b4eb2d5cc044c32560b3e5cfc8b878078d9ad77f
Author: Tom Lane <tgl@sss.pgh.pa.us>
Date: Mon Apr 21 17:28:13 2014 +0000
First released in: 9.4.0, 9.2.9, 9.3.5
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.
contrib/pg_stat_statements/pg_stat_statements.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/contrib/pg_stat_statements/pg_stat_statements.c b/contrib/pg_stat_statements/pg_stat_statements.c
index aa11c144d68..a41434b046d 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);
[parent: c6b55bec3fb2]