]> granicus.if.org Git - postgresql/commit
Fix SPI documentation for new handling of ExecutorRun's count parameter.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 24 Jan 2013 23:34:00 +0000 (18:34 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 24 Jan 2013 23:34:00 +0000 (18:34 -0500)
commit2ddc600f8f0252a0864e85d5cc1eeb3b9687d7e9
tree3df9d556becd674b06e19df2d3e3251c9a98fcf2
parent1068771abfeec148a9c1ce4782785bddc1982070
Fix SPI documentation for new handling of ExecutorRun's count parameter.

Since 9.0, the count parameter has only limited the number of tuples
actually returned by the executor.  It doesn't affect the behavior of
INSERT/UPDATE/DELETE unless RETURNING is specified, because without
RETURNING, the ModifyTable plan node doesn't return control to execMain.c
for each tuple.  And we only check the limit at the top level.

While this behavioral change was unintentional at the time, discussion of
bug #6572 led us to the conclusion that we prefer the new behavior anyway,
and so we should just adjust the docs to match rather than change the code.
Accordingly, do that.  Back-patch as far as 9.0 so that the docs match the
code in each branch.
doc/src/sgml/spi.sgml
src/backend/executor/execMain.c