Re: Enhanced containment selectivity function

Started by Matteo Beccatiabout 20 years ago3 messageshackers
Jump to latest
#1Matteo Beccati
php@beccati.com

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.

Best regards
--
Matteo Beccati
http://phpadsnew.com
http://phppgads.com

#2Bruce Momjian
bruce@momjian.us
In reply to: Matteo Beccati (#1)

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.

Yes, I talked to Tom today about moving that stuff into /contrib. I
will work on the patch to do that.

--
Bruce Momjian http://candle.pha.pa.us
SRA OSS, Inc. http://www.sraoss.com

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

#3Bruce Momjian
bruce@momjian.us
In reply to: Matteo Beccati (#1)
Re: [HACKERS] Enhanced containment selectivity function

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. +

Attachments:

/bjm/difftext/x-patchDownload+242-234