postgres.git / summary / log / commit / refs

commit    053959397bd1f5e10d16d68bb91b7beffcb673e7
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Mon Apr 21 17:28:10 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 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);
 


[parent: 98876f0ff18e]