Thread: pgsql: Dept of second thoughts: let's make sure that

pgsql: Dept of second thoughts: let's make sure that

From
tgl@postgresql.org (Tom Lane)
Date:
Log Message:
-----------
Dept of second thoughts: let's make sure that get_index_stats_hook is only
applied to expression indexes, not to plain relations.  The original coding
in btcostestimate conflated the two cases, but it's not hard to use
get_relation_stats_hook instead when we're looking to the underlying relation.

Modified Files:
--------------
    pgsql/src/backend/utils/adt:
        selfuncs.c (r1.254 -> r1.255)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/selfuncs.c?r1=1.254&r2=1.255)

Re: pgsql: Dept of second thoughts: let's make sure that

From
Simon Riggs
Date:
On Sun, 2008-09-28 at 20:42 +0000, Tom Lane wrote:
> Log Message:
> -----------
> Dept of second thoughts: let's make sure that get_index_stats_hook is only
> applied to expression indexes, not to plain relations.  The original coding
> in btcostestimate conflated the two cases, but it's not hard to use
> get_relation_stats_hook instead when we're looking to the underlying relation.

Good catch, thanks.

--
 Simon Riggs           www.2ndQuadrant.com
 PostgreSQL Training, Services and Support