Re: [HACKERS] Enhanced containment selectivity function

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Matteo Beccati <php(at)beccati(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Oleg Bartunov <oleg(at)sai(dot)msu(dot)su>, Teodor Sigaev <teodor(at)sigaev(dot)ru>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [HACKERS] Enhanced containment selectivity function
Date: 2006-04-26 22:32:01
Message-ID: 200604262232.k3QMW1018118@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Matteo Beccati wrote:
> Bruce Momjian ha scritto:
> > Your patch has been added to the PostgreSQL unapplied patches list at:
> >
> > http://momjian.postgresql.org/cgi-bin/pgpatches
> >
> > It will be applied as soon as one of the PostgreSQL committers reviews
> > and approves it.
>
> Great. I would just like to remind that Tom said:
>
> > I'd be willing to consider exporting those functions from selfuncs.c.
>
> so that the selector function could be moved to contrib/ltree, which is
> its natural place.
>
> It could also be noted that a similar feature could be useful outside
> ltree: I guess there are plenty of cases when scanning statistics would
> give a better result than using a constant selectivity.

I have done as you and Tom suggested, moving parentsel into
/contrib/ltree. Patch attached and applied, catalog version attached.

--
Bruce Momjian http://candle.pha.pa.us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

Attachment Content-Type Size
/bjm/diff text/x-patch 21.7 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Jim C. Nasby 2006-04-26 22:34:11 Re: Catalog Access (was: [GENERAL] Concurrency problem
Previous Message Wes 2006-04-26 22:24:27 Re: Catalog Access (was: [GENERAL] Concurrency problem

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2006-04-26 22:36:26 Re: [HACKERS] Enhanced containment selectivity function
Previous Message Magnus Hagander 2006-04-26 22:07:39 Updated VC++ patch