]> granicus.if.org Git - postgresql/commitdiff
Fix executor/README to reflect disallowing SRFs in UPDATE.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 13 Sep 2016 18:25:35 +0000 (14:25 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 13 Sep 2016 18:25:35 +0000 (14:25 -0400)
The parenthetical comment here is obsoleted by commit a4c35ea1c.
Noted by Andres Freund.

src/backend/executor/README

index 8afa1e3e4a7596475cbf19a76c88d48a04aeef02..f1d1e4c76ce9d78cfc70bc6da7348e2a03dae52d 100644 (file)
@@ -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.