From 5afb57038f338adefb282afb7976c3ab86a92085 Mon Sep 17 00:00:00 2001 From: Amit Kapila Date: Fri, 7 Jun 2019 05:45:06 +0530 Subject: [PATCH] Fix inconsistency in comments atop ExecParallelEstimate. When this code was initially introduced in commit d1b7c1ff, the structure used was SharedPlanStateInstrumentation, but later when it got changed to Instrumentation structure in commit b287df70, we forgot to update the comment. Reported-by: Wu Fei Author: Wu Fei Reviewed-by: Amit Kapila Backpatch-through: 9.6 Discussion: https://postgr.es/m/52E6E0843B9D774C8C73D6CF64402F0562215EB2@G08CNEXMBPEKD02.g08.fujitsu.local --- src/backend/executor/execParallel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/backend/executor/execParallel.c b/src/backend/executor/execParallel.c index b1a3d91a196..47724080c9e 100644 --- a/src/backend/executor/execParallel.c +++ b/src/backend/executor/execParallel.c @@ -172,7 +172,7 @@ ExecSerializePlan(Plan *plan, EState *estate) * shm_toc_estimate_keys on &pcxt->estimator. * * While we're at it, count the number of PlanState nodes in the tree, so - * we know how many SharedPlanStateInstrumentation structures we need. + * we know how many Instrumentation structures we need. */ static bool ExecParallelEstimate(PlanState *planstate, ExecParallelEstimateContext *e)