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)
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 |