Fix inconsistency in comments atop ExecParallelEstimate.

Amit Kapila · Amit Kapila · commit d8261595bc6f · 2019-06-07T05:23:52.000+05:30
When this code was initially introduced in commit d1b7c1f, the structure used was SharedPlanStateInstrumentation, but later when it got changed to Instrumentation structure in commit b287df7, 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
diff --git 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)

Original file line number	Diff line number	Diff line change
`@@ -219,7 +219,7 @@ ExecSerializePlan(Plan plan, EState estate)`
`219`	`219`	`* &pcxt->estimator.`
`220`	`220`	`*`
`221`	`221`	`* While we're at it, count the number of PlanState nodes in the tree, so`
`222`		`- * we know how many SharedPlanStateInstrumentation structures we need.`
	`222`	`+ * we know how many Instrumentation structures we need.`
`223`	`223`	`*/`
`224`	`224`	`static bool`
`225`	`225`	`ExecParallelEstimate(PlanState planstate, ExecParallelEstimateContext e)`