Thread: pgsql: Dept of second thoughts: let's make sure that
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)
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