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