diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2014-12-11 19:37:00 -0500 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2014-12-11 19:37:36 -0500 |
commit | 2db576ba8c449fcaf61ae7aa14ed62e63ebf5924 (patch) | |
tree | b52d0c09ff79a8852ae6c0280c68cb9a6b80cab3 /contrib/postgres_fdw/postgres_fdw.c | |
parent | 2646d2d4a9d8b978fda0d0d1affae4788a30f604 (diff) | |
download | postgresql-2db576ba8c449fcaf61ae7aa14ed62e63ebf5924.tar.gz postgresql-2db576ba8c449fcaf61ae7aa14ed62e63ebf5924.zip |
Fix corner case where SELECT FOR UPDATE could return a row twice.
In READ COMMITTED mode, if a SELECT FOR UPDATE discovers it has to redo
WHERE-clause checking on rows that have been updated since the SELECT's
snapshot, it invokes EvalPlanQual processing to do that. If this first
occurs within a non-first child table of an inheritance tree, the previous
coding could accidentally re-return a matching row from an earlier,
already-scanned child table. (And, to add insult to injury, I think this
could make it miss returning a row that should have been returned, if the
updated row that this happens on should still have passed the WHERE qual.)
Per report from Kyotaro Horiguchi; the added isolation test is based on his
test case.
This has been broken for quite awhile, so back-patch to all supported
branches.
Diffstat (limited to 'contrib/postgres_fdw/postgres_fdw.c')
0 files changed, 0 insertions, 0 deletions