diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2014-12-12 12:41:49 -0500 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2014-12-12 12:41:49 -0500 |
commit | 8ec8760fc87ecde0516e511f1c55aec627b01ea7 (patch) | |
tree | 5c7130c19b17f0b9e5f56f1d64969e961bdb6d54 /src/backend/utils/adt/jsonb.c | |
parent | 237a8824430c607fce1eafde0c625744d50a455c (diff) | |
download | postgresql-8ec8760fc87ecde0516e511f1c55aec627b01ea7.tar.gz postgresql-8ec8760fc87ecde0516e511f1c55aec627b01ea7.zip |
Revert misguided change to postgres_fdw FOR UPDATE/SHARE code.
In commit 462bd95705a0c23ba0b0ba60a78d32566a0384c1, I changed postgres_fdw
to rely on get_plan_rowmark() instead of get_parse_rowmark(). I still
think that's a good idea in the long run, but as Etsuro Fujita pointed out,
it doesn't work today because planner.c forces PlanRowMarks to have
markType = ROW_MARK_COPY for all foreign tables. There's no urgent reason
to change this in the back branches, so let's just revert that part of
yesterday's commit rather than trying to design a better solution under
time pressure.
Also, add a regression test case showing what postgres_fdw does with FOR
UPDATE/SHARE. I'd blithely assumed there was one already, else I'd have
realized yesterday that this code didn't work.
Diffstat (limited to 'src/backend/utils/adt/jsonb.c')
0 files changed, 0 insertions, 0 deletions