pgsql: Partially revert my patch of 2008-11-12 that installed a limit on

From: tgl(at)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Partially revert my patch of 2008-11-12 that installed a limit on
Date: 2009-05-11 17:56:08
Message-ID: 20090511175608.A9311754067@cvs.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Log Message:
-----------
Partially revert my patch of 2008-11-12 that installed a limit on the number
of AND/OR clause branches that predtest.c would attempt to deal with. As
noted in bug #4721, that change disabled proof attempts for sizes of problems
that people are actually expecting it to work for. The original complaint
it was trying to solve was O(N^2) behavior for long IN-lists, so let's try
applying the limit to just ScalarArrayOpExprs rather than everything.
Another case of "foolish consistency" I fear.

Back-patch to 8.2, same as the previous patch was.

Modified Files:
--------------
pgsql/src/backend/optimizer/util:
predtest.c (r1.25 -> r1.26)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/util/predtest.c?r1=1.25&r2=1.26)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2009-05-11 17:56:14 pgsql: Partially revert my patch of 2008-11-12 that installed a limit on
Previous Message User H-saito 2009-05-11 15:16:57 pg-migrator - pg_migrator: Oops,parameter difference.