pgsql: Apply a band-aid fix for the problem that 8.2 and up completely - Mailing list pgsql-committers

From tgl@postgresql.org (Tom Lane)
Subject pgsql: Apply a band-aid fix for the problem that 8.2 and up completely
Date
Msg-id 20070831233530.274DA754201@cvs.postgresql.org
Whole thread Raw
List pgsql-committers
Log Message:
-----------
Apply a band-aid fix for the problem that 8.2 and up completely misestimate
the number of rows likely to be produced by a query such as
    SELECT * FROM t1 LEFT JOIN t2 USING (key) WHERE t2.key IS NULL;
What this is doing is selecting for t1 rows with no match in t2, and thus
it may produce a significant number of rows even if the t2.key table column
contains no nulls at all.  8.2 thinks the table column's null fraction is
relevant and thus may estimate no rows out, which results in terrible plans
if there are more joins above this one.  A proper fix for this will involve
passing much more information about the context of a clause to the selectivity
estimator functions than we ever have.  There's no time left to write such a
patch for 8.3, and it wouldn't be back-patchable into 8.2 anyway.  Instead,
put in an ad-hoc test to defeat the normal table-stats-based estimation when
an IS NULL test is evaluated at an outer join, and just use a constant
estimate instead --- I went with 0.5 for lack of a better idea.  This won't
catch every case but it will catch the typical ways of writing such queries,
and it seems unlikely to make things worse for other queries.

Tags:
----
REL8_2_STABLE

Modified Files:
--------------
    pgsql/src/backend/optimizer/path:
        clausesel.c (r1.82 -> r1.82.2.1)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/path/clausesel.c?r1=1.82&r2=1.82.2.1)
    pgsql/src/backend/utils/adt:
        selfuncs.c (r1.214.2.5 -> r1.214.2.6)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/selfuncs.c?r1=1.214.2.5&r2=1.214.2.6)
    pgsql/src/include/utils:
        selfuncs.h (r1.36 -> r1.36.2.1)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/utils/selfuncs.h?r1=1.36&r2=1.36.2.1)

pgsql-committers by date:

Previous
From: tgl@postgresql.org (Tom Lane)
Date:
Subject: pgsql: Apply a band-aid fix for the problem that 8.2 and up completely
Next
From: hinoue@pgfoundry.org (User Hinoue)
Date:
Subject: psqlodbc - psqlodbc: The version is now 8.2.0403.