Thread: Patch to contrib/fulltextindex/fti.sql

Patch to contrib/fulltextindex/fti.sql

From
"Christopher Kings-Lynne"
Date:
The fti.pl supplied with the fulltextindex module generate ALL possible
substrings of two characters or greater, and is case-sensitive.

This patch makes it work correctly.  It generates only the suffixes of each
word, plus lowercases them - as specified by the README file.

This brings it into line with the fti.c function, makes it case-insensitive
properly, removes the problem with duplicate rows being returned from an fti
search and greatly reduces the size of the generated index table.

It was written by my co-worker, Brett Toolin.

Chris

Attachment

Re: Patch to contrib/fulltextindex/fti.sql

From
Bruce Momjian
Date:
Your patch has been added to the PostgreSQL unapplied patches list at:

    http://candle.pha.pa.us/cgi-bin/pgpatches

I will try to apply it within the next 48 hours.

> The fti.pl supplied with the fulltextindex module generate ALL possible
> substrings of two characters or greater, and is case-sensitive.
>
> This patch makes it work correctly.  It generates only the suffixes of each
> word, plus lowercases them - as specified by the README file.
>
> This brings it into line with the fti.c function, makes it case-insensitive
> properly, removes the problem with duplicate rows being returned from an fti
> search and greatly reduces the size of the generated index table.
>
> It was written by my co-worker, Brett Toolin.
>
> Chris

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 6: Have you searched our list archives?
>
> http://www.postgresql.org/search.mpl

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

Re: Patch to contrib/fulltextindex/fti.sql

From
Bruce Momjian
Date:
Thanks.  Patch applied.


> The fti.pl supplied with the fulltextindex module generate ALL possible
> substrings of two characters or greater, and is case-sensitive.
>
> This patch makes it work correctly.  It generates only the suffixes of each
> word, plus lowercases them - as specified by the README file.
>
> This brings it into line with the fti.c function, makes it case-insensitive
> properly, removes the problem with duplicate rows being returned from an fti
> search and greatly reduces the size of the generated index table.
>
> It was written by my co-worker, Brett Toolin.
>
> Chris

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 6: Have you searched our list archives?
>
> http://www.postgresql.org/search.mpl

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

Re: Patch to contrib/fulltextindex/fti.sql

From
Bruce Momjian
Date:
Patch applied.  Thanks.

> The fti.pl supplied with the fulltextindex module generate ALL possible
> substrings of two characters or greater, and is case-sensitive.
>
> This patch makes it work correctly.  It generates only the suffixes of each
> word, plus lowercases them - as specified by the README file.
>
> This brings it into line with the fti.c function, makes it case-insensitive
> properly, removes the problem with duplicate rows being returned from an fti
> search and greatly reduces the size of the generated index table.
>
> It was written by my co-worker, Brett Toolin.
>
> Chris

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 6: Have you searched our list archives?
>
> http://www.postgresql.org/search.mpl

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026