Adjust ExplainOneQuery_hook_type to take a DestReceiver argument.

robertmhaas · robertmhaas · commit 0bf42a5f3b62 · 2013-04-09T10:25:08.000-04:00
The materialized views patch adjusted ExplainOneQuery to take an
additional DestReceiver argument, but failed to add a matching
argument to the definition of ExplainOneQuery_hook.  This is a
problem for users of the hook that want to call ExplainOnePlan.
Fix by adding the missing argument.
diff --git a/src/backend/commands/explain.c b/src/backend/commands/explain.c
@@ -312,7 +312,7 @@ ExplainOneQuery(Query *query, IntoClause *into, ExplainState *es,
 
 	/* if an advisor plugin is present, let it manage things */
 	if (ExplainOneQuery_hook)
-		(*ExplainOneQuery_hook) (query, into, es, queryString, params);
+		(*ExplainOneQuery_hook) (query, into, es, queryString, dest, params);
 	else
 	{
 		PlannedStmt *plan;
diff --git a/src/include/commands/explain.h b/src/include/commands/explain.h
@@ -47,6 +47,7 @@ typedef void (*ExplainOneQuery_hook_type) (Query *query,
 													   IntoClause *into,
 													   ExplainState *es,
 													 const char *queryString,
+													   DestReceiver *dest,
 													   ParamListInfo params);
 extern PGDLLIMPORT ExplainOneQuery_hook_type ExplainOneQuery_hook;
 

Original file line number	Diff line number	Diff line change
`@@ -312,7 +312,7 @@ ExplainOneQuery(Query query, IntoClause into, ExplainState *es,`
`312`	`312`
`313`	`313`	`/* if an advisor plugin is present, let it manage things */`
`314`	`314`	`if (ExplainOneQuery_hook)`
`315`		`- (*ExplainOneQuery_hook) (query, into, es, queryString, params);`
	`315`	`+ (*ExplainOneQuery_hook) (query, into, es, queryString, dest, params);`
`316`	`316`	`else`
`317`	`317`	`{`
`318`	`318`	`PlannedStmt *plan;`