postgres.git / summary / log / commit / refs

commit    78a3c9b6a5f6cc84abaf4e13deb58c620eb2161b
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Mon Apr 21 17:28:07 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 2ad3fb58868..de82ef5a1f5 100644
--- a/contrib/pg_stat_statements/pg_stat_statements.c
+++ b/contrib/pg_stat_statements/pg_stat_statements.c
@@ -767,6 +767,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: 602b27ab8e45]