Fix missing space in EXPLAIN ANALYZE output.

robertmhaas · robertmhaas · commit 51d3e279c3e1 · 2025-02-28T13:04:12.000-05:00
Commit ddb17e3 introduced this regression. Ideally, the regression tests would have caught this mistake, but apparently they don't test with timing enabled, presumably because that would make the output vary. Author: Thom Brown <thom@linux.com> Reviewed-by: Fabrízio de Royes Mello <fabriziomello@gmail.com> Discussion: http://postgr.es/m/CAA-aLv6nq=UeiyvM7_Mxgo9TVBzs2oh46b9vfyLzuyVEz3j1-g@mail.gmail.com
diff --git a/src/backend/commands/explain.c b/src/backend/commands/explain.c
@@ -2038,7 +2038,7 @@ ExplainNode(PlanState *planstate, List *ancestors,
 				ExplainIndentText(es);
 				appendStringInfo(es->str, "actual ");
 				if (es->timing)
-					appendStringInfo(es->str, "time=%.3f..%.3f", startup_ms, total_ms);
+					appendStringInfo(es->str, "time=%.3f..%.3f ", startup_ms, total_ms);
 
 				appendStringInfo(es->str, "rows=%.2f loops=%.0f\n", rows, nloops);
 			}

Original file line number	Diff line number	Diff line change
`@@ -2038,7 +2038,7 @@ ExplainNode(PlanState planstate, List ancestors,`
`2038`	`2038`	`ExplainIndentText(es);`
`2039`	`2039`	`appendStringInfo(es->str, "actual ");`
`2040`	`2040`	`if (es->timing)`
`2041`		`- appendStringInfo(es->str, "time=%.3f..%.3f", startup_ms, total_ms);`
	`2041`	`+ appendStringInfo(es->str, "time=%.3f..%.3f ", startup_ms, total_ms);`
`2042`	`2042`
`2043`	`2043`	`appendStringInfo(es->str, "rows=%.2f loops=%.0f\n", rows, nloops);`
`2044`	`2044`	`}`