From d8261595bc6f5189896cdce6861915ac23830e7c Mon Sep 17 00:00:00 2001 From: Amit Kapila Date: Fri, 7 Jun 2019 05:23:52 +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 0ab9a9939c..53cd2fc666 100644 --- a/src/backend/executor/execParallel.c +++ b/src/backend/executor/execParallel.c @@ -219,7 +219,7 @@ ExecSerializePlan(Plan *plan, EState *estate) * &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) -- 2.40.0