From d5e3fe682ac327ae2c0682a931f95e946ae13487 Mon Sep 17 00:00:00 2001 From: Etsuro Fujita Date: Thu, 29 Sep 2022 16:55:00 +0900 Subject: [PATCH] Update comment in ExecInsert() regarding batch insertion. 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 --- src/backend/executor/nodeModifyTable.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/backend/executor/nodeModifyTable.c b/src/backend/executor/nodeModifyTable.c index 775960827d..04454ad6e6 100644 --- 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) {