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

Started by Tom Lanealmost 18 years ago2 messagescomitters
Jump to latest
#1Tom Lane
tgl@sss.pgh.pa.us

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)

#2Simon Riggs
simon@2ndQuadrant.com
In reply to: Tom Lane (#1)
Re: pgsql: Dept of second thoughts: let's make sure that

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