From: | Cédric Villemain <cedric(dot)villemain(at)dalibo(dot)com> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org |
Cc: | Oleg Bartunov <oleg(at)sai(dot)msu(dot)su>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Teodor Sigaev <teodor(at)sigaev(dot)ru> |
Subject: | Re: tsearch2 patch status report |
Date: | 2007-08-21 17:03:13 |
Message-ID: | 200708211903.14217.cedric.villemain@dalibo.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Le mardi 21 août 2007, Oleg Bartunov a écrit :
> On Mon, 20 Aug 2007, Tom Lane wrote:
> > I've applied version 0.58 of the patch with a lot of further
> > editorializing. I feel fairly confident now in the code that interfaces
>
> Great ! Just checked and most things after trivial changes are working !
> We need to summarize changes and provide upgraide guide.
Congratulations Teodor and Oleg !
>
> > Also, we need to decide what to do with contrib/tsearch2, which is
> > currently DOA because of conflicts with the new core code. We could
> > either rip it out entirely, or try to convert it into a compatibility
> > package. In view of the renamings of functions we agreed to do, I
> > think there is some scope for a compatibility package, but I have no
> > time to work on that.
>
> Probably, we could leave tsearch2 in contrib as a compat module and
> explicitly define 8.4 will be the last release with tsearch2 support.
Are you going to keep tools to compile stemmer from tsearch2 ? (I think about
those files :
http://developer.postgresql.org/cvsweb.cgi/pgsql/contrib/tsearch2/gendict/config.sh )
Or just let people use this method :
http://momjian.us/expire/textsearch/HTML/textsearch-parser-example.html ?
--
Cédric Villemain
Administrateur de Base de Données
Cel: +33 (0)6 74 15 56 53
http://dalibo.com - http://dalibo.org
From | Date | Subject | |
---|---|---|---|
Next Message | Heikki Linnakangas | 2007-08-21 17:05:40 | Re: tsearch2 patch status report |
Previous Message | Tom Lane | 2007-08-21 17:03:06 | Re: tsearch2 patch status report |