From: Tom Lane Date: Tue, 13 Sep 2016 18:25:35 +0000 (-0400) Subject: Fix executor/README to reflect disallowing SRFs in UPDATE. X-Git-Tag: REL_10_BETA1~1722 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=fdc79e1909dc3866a385ffb74bdd6ce6a082a300;p=postgresql Fix executor/README to reflect disallowing SRFs in UPDATE. The parenthetical comment here is obsoleted by commit a4c35ea1c. Noted by Andres Freund. --- diff --git a/src/backend/executor/README b/src/backend/executor/README index 8afa1e3e4a..f1d1e4c76c 100644 --- a/src/backend/executor/README +++ b/src/backend/executor/README @@ -195,8 +195,7 @@ the entire row value in the join output row. We disallow set-returning functions in the targetlist of SELECT FOR UPDATE, so as to ensure that at most one tuple can be returned for any particular set of scan tuples. Otherwise we'd get duplicates due to the original -query returning the same set of scan tuples multiple times. (Note: there -is no explicit prohibition on SRFs in UPDATE, but the net effect will be -that only the first result row of an SRF counts, because all subsequent -rows will result in attempts to re-update an already updated target row. -This is historical behavior and seems not worth changing.) +query returning the same set of scan tuples multiple times. Likewise, +SRFs are disallowed in an UPDATE's targetlist. There, they would have the +effect of the same row being updated multiple times, which is not very +useful --- and updates after the first would have no effect anyway.