pgsql: Tighten up SS_finalize_plan's computation of valid_params to

From: tgl(at)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Tighten up SS_finalize_plan's computation of valid_params to
Date: 2008-07-10 02:14:03
Message-ID: 20080710021403.6AF2D754857@cvs.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Log Message:
-----------
Tighten up SS_finalize_plan's computation of valid_params to exclude Params of
the current query level that aren't in fact output parameters of the current
initPlans. (This means, for example, output parameters of regular subplans.)
To make this work correctly for output parameters coming from sibling
initplans requires rejiggering the API of SS_finalize_plan just a bit:
we need the siblings to be visible to it, rather than hidden as
SS_make_initplan_from_plan had been doing. This is really part of my response
to bug #4290, but I concluded this part probably shouldn't be back-patched,
since all that it's doing is to make a debugging cross-check tighter.

Modified Files:
--------------
pgsql/src/backend/optimizer/plan:
planagg.c (r1.40 -> r1.41)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/plan/planagg.c?r1=1.40&r2=1.41)
planner.c (r1.233 -> r1.234)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/plan/planner.c?r1=1.233&r2=1.234)
subselect.c (r1.131 -> r1.132)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/plan/subselect.c?r1=1.131&r2=1.132)
pgsql/src/include/optimizer:
subselect.h (r1.30 -> r1.31)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/optimizer/subselect.h?r1=1.30&r2=1.31)

Browse pgsql-committers by date

  From Date Subject
Next Message User Fxjr 2008-07-10 05:10:44 npgsql - Npgsql2: Fixed a problem where LargeObjectManager was
Previous Message Tom Lane 2008-07-10 01:17:52 pgsql: Fix mis-calculation of extParam/allParam sets for plan nodes, as