pgsql: Tweak distribute_qual_to_rels so that when we decide a

From: tgl(at)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Tweak distribute_qual_to_rels so that when we decide a
Date: 2009-05-06 20:31:18
Message-ID: 20090506203118.EACD4754069@cvs.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Log Message:
-----------
Tweak distribute_qual_to_rels so that when we decide a pseudoconstant qual
can be pushed to the top of the join tree, we update both the relids and
qualscope variables to keep them in sync. This prevents a possible later
failure of an Assert clause, and affects nothing else since qualscope isn't
used later except for that Assert. At the moment the Assert shouldn't be
reachable when we've pushed the qual up; but this is cheap insurance, and
it's more sensible anyway in terms of the overall logic of the routine.
Per analysis of a bug report from Stefan Huehner.

I'm not back-patching this since it's just future-proofing; but if anyone
gets tempted to change check_outerjoin_delay again in the back branches,
this might be needed.

Modified Files:
--------------
pgsql/src/backend/optimizer/plan:
initsplan.c (r1.151 -> r1.152)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/plan/initsplan.c?r1=1.151&r2=1.152)

Browse pgsql-committers by date

  From Date Subject
Next Message User Itagaki 2009-05-07 01:11:02 orafce - orafce: PG_EUC_JIS_2004 is an enum value, not a macro.
Previous Message Tom Lane 2009-05-06 16:15:21 pgsql: Modify CREATE DATABASE to enforce that the source database's