From 21320763a39994021a4ff045ff66f7204bdb6d78 Mon Sep 17 00:00:00 2001 From: Andrew Dunstan Date: Mon, 13 Feb 2012 08:22:15 -0500 Subject: [PATCH] Fix auto-explain JSON output to be valid JSON. Problem reported by Peter Eisentraut. Backpatched to release 9.0. --- contrib/auto_explain/auto_explain.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/contrib/auto_explain/auto_explain.c b/contrib/auto_explain/auto_explain.c index 9fc0ae1d92..e48ea489dc 100644 --- a/contrib/auto_explain/auto_explain.c +++ b/contrib/auto_explain/auto_explain.c @@ -299,6 +299,13 @@ explain_ExecutorEnd(QueryDesc *queryDesc) if (es.str->len > 0 && es.str->data[es.str->len - 1] == '\n') es.str->data[--es.str->len] = '\0'; + /* Fix JSON to output an object */ + if (auto_explain_log_format == EXPLAIN_FORMAT_JSON) + { + es.str->data[0] = '{'; + es.str->data[es.str->len - 1] = '}'; + } + /* * Note: we rely on the existing logging of context or * debug_query_string to identify just which statement is being