Update comment in ExecInsert() regarding batch insertion.

Etsuro Fujita · Etsuro Fujita · commit d5e3fe682ac3 · 2022-09-29T16:55:00.000+09:00
Remove the stale text that is a leftover from an earlier version of the patch to add support for batch insertion, and adjust the wording in the remaining text. Back-patch to v14 where batch insertion came in. Review and wording adjustment by Tom Lane. Discussion: https://postgr.es/m/CAPmGK14goatHPHQv2Aeu_UTKqZ%2BBO%2BP%2Bzd3HKv5D%2BdyyfWKDSw%40mail.gmail.com
diff --git a/src/backend/executor/nodeModifyTable.c b/src/backend/executor/nodeModifyTable.c
@@ -795,9 +795,8 @@ ExecInsert(ModifyTableContext *context,
 		if (resultRelInfo->ri_BatchSize > 1)
 		{
 			/*
-			 * If a certain number of tuples have already been accumulated, or
-			 * a tuple has come for a different relation than that for the
-			 * accumulated tuples, perform the batch insert
+			 * When we've reached the desired batch size, perform the
+			 * insertion.
 			 */
 			if (resultRelInfo->ri_NumSlots == resultRelInfo->ri_BatchSize)
 			{

Original file line number	Diff line number	Diff line change
`@@ -795,9 +795,8 @@ ExecInsert(ModifyTableContext *context,`
`795`	`795`	`if (resultRelInfo->ri_BatchSize > 1)`
`796`	`796`	`{`
`797`	`797`	`/*`
`798`		`- * If a certain number of tuples have already been accumulated, or`
`799`		`- * a tuple has come for a different relation than that for the`
`800`		`- * accumulated tuples, perform the batch insert`
	`798`	`+ * When we've reached the desired batch size, perform the`
	`799`	`+ * insertion.`
`801`	`800`	`*/`
`802`	`801`	`if (resultRelInfo->ri_NumSlots == resultRelInfo->ri_BatchSize)`
`803`	`802`	`{`