Thread: pgsql: Make SELECT FOR UPDATE/SHARE work on inheritance trees, by having

pgsql: Make SELECT FOR UPDATE/SHARE work on inheritance trees, by having

From
tgl@postgresql.org (Tom Lane)
Date:
Log Message:
-----------
Make SELECT FOR UPDATE/SHARE work on inheritance trees, by having the plan
return the tableoid as well as the ctid for any FOR UPDATE targets that
have child tables.  All child tables are listed in the ExecRowMark list,
but the executor just skips the ones that didn't produce the current row.

Curiously, this longstanding restriction doesn't seem to have been documented
anywhere; so no doc changes.

Modified Files:
--------------
    pgsql/src/backend/executor:
        execMain.c (r1.315 -> r1.316)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/execMain.c?r1=1.315&r2=1.316)
    pgsql/src/backend/nodes:
        copyfuncs.c (r1.411 -> r1.412)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/copyfuncs.c?r1=1.411&r2=1.412)
        equalfuncs.c (r1.336 -> r1.337)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/equalfuncs.c?r1=1.336&r2=1.337)
        outfuncs.c (r1.344 -> r1.345)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/outfuncs.c?r1=1.344&r2=1.345)
        readfuncs.c (r1.216 -> r1.217)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/nodes/readfuncs.c?r1=1.216&r2=1.217)
    pgsql/src/backend/optimizer/path:
        allpaths.c (r1.176 -> r1.177)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/path/allpaths.c?r1=1.176&r2=1.177)
    pgsql/src/backend/optimizer/prep:
        preptlist.c (r1.93 -> r1.94)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/prep/preptlist.c?r1=1.93&r2=1.94)
        prepunion.c (r1.161 -> r1.162)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/prep/prepunion.c?r1=1.161&r2=1.162)
    pgsql/src/backend/parser:
        analyze.c (r1.382 -> r1.383)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/parser/analyze.c?r1=1.382&r2=1.383)
    pgsql/src/backend/rewrite:
        rewriteManip.c (r1.117 -> r1.118)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/rewrite/rewriteManip.c?r1=1.117&r2=1.118)
    pgsql/src/include/catalog:
        catversion.h (r1.507 -> r1.508)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/catalog/catversion.h?r1=1.507&r2=1.508)
    pgsql/src/include/nodes:
        execnodes.h (r1.194 -> r1.195)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/nodes/execnodes.h?r1=1.194&r2=1.195)
        parsenodes.h (r1.377 -> r1.378)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/nodes/parsenodes.h?r1=1.377&r2=1.378)
    pgsql/src/test/regress/expected:
        portals.out (r1.17 -> r1.18)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/expected/portals.out?r1=1.17&r2=1.18)
    pgsql/src/test/regress/sql:
        portals.sql (r1.14 -> r1.15)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/sql/portals.sql?r1=1.14&r2=1.15)

Re: pgsql: Make SELECT FOR UPDATE/SHARE work on inheritance trees, by having

From
Bruce Momjian
Date:
Tom Lane wrote:
> Log Message:
> -----------
> Make SELECT FOR UPDATE/SHARE work on inheritance trees, by having the plan
> return the tableoid as well as the ctid for any FOR UPDATE targets that
> have child tables.  All child tables are listed in the ExecRowMark list,
> but the executor just skips the ones that didn't produce the current row.
>
> Curiously, this longstanding restriction doesn't seem to have been documented
> anywhere; so no doc changes.

It was documented on the TODO list;  marked as done now.

--
  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +