Clamp indexscan filter condition cost estimate to be not less than zero.

cost_index tries to estimate the per-tuple costs of evaluating filter
conditions (a/k/a qpquals) by subtracting the estimated cost of the
indexqual conditions from that of the baserestrictinfo conditions.  This is
correct so long as the indexquals list is a subset of the baserestrictinfo
list.  However, in the presence of derived indexable conditions it's
completely wrong, leading to bogus or even negative scan cost estimates,
as seen for example in bug #6579 from Istvan Endredy.  In practice the
problem isn't severe except in the specific case of a LIKE optimization on
a functional index containing a very expensive function.

A proper fix for this might change cost estimates by more than people would
like for stable branches, so in the back branches let's just clamp the cost
difference to be not less than zero.  That will at least prevent completely
insane behavior, while not changing the results normally.

diff --git a/src/backend/optimizer/path/costsize.c b/src/backend/optimizer/path/costsize.c
index c853bd8dcb6ca6758b6b515abea577712aa07889..94e42a8a1467c56937082f343f02934829717f2e 100644 (file)
--- a/src/backend/optimizer/path/costsize.c
+++ b/src/backend/optimizer/path/costsize.c
@@ -392,6 +392,14 @@ cost_index(IndexPath *path, PlannerInfo *root,
         * some of the indexquals are join clauses and shouldn't be subtracted.
         * Rather than work out exactly how much to subtract, we don't subtract
         * anything.
+        *
+        * XXX actually, this calculation is almost completely bogus, because
+        * indexquals will contain derived indexable conditions which might be
+        * quite different from the "original" quals in baserestrictinfo.  We
+        * ought to determine the actual qpqual list and cost that, rather than
+        * using this shortcut.  But that's too invasive a change to consider
+        * back-patching, so for the moment we just mask the worst aspects of the
+        * problem by clamping the subtracted amount.
         */
        startup_cost += baserel->baserestrictcost.startup;
        cpu_per_tuple = cpu_tuple_cost + baserel->baserestrictcost.per_tuple;
@@ -402,7 +410,8 @@ cost_index(IndexPath *path, PlannerInfo *root,
 
                cost_qual_eval(&index_qual_cost, indexQuals, root);
                /* any startup cost still has to be paid ... */
-               cpu_per_tuple -= index_qual_cost.per_tuple;
+               cpu_per_tuple -= Min(index_qual_cost.per_tuple,
+                                                        baserel->baserestrictcost.per_tuple);
        }
 
        run_cost += cpu_per_tuple * tuples_fetched;