Skip to content

Commit

Permalink
Update comment in ExecInsert() regarding batch insertion.
Browse files Browse the repository at this point in the history
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
  • Loading branch information
Etsuro Fujita committed Sep 29, 2022
1 parent 81c094b commit fc9eb3f
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions src/backend/executor/nodeModifyTable.c
Expand Up @@ -679,9 +679,8 @@ ExecInsert(ModifyTableState *mtstate,
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)
{
Expand Down

0 comments on commit fc9eb3f

Please sign in to comment.