Re: Preliminary patch for tsearch example dictionaries/parsers in contrib - Mailing list pgsql-patches

From Tom Lane
Subject Re: Preliminary patch for tsearch example dictionaries/parsers in contrib
Date
Msg-id 14958.1192023222@sss.pgh.pa.us
Whole thread Raw
In response to Re: Preliminary patch for tsearch example dictionaries/parsers in contrib  (Oleg Bartunov <oleg@sai.msu.su>)
Responses Re: Preliminary patch for tsearch example dictionaries/parsers in contrib
List pgsql-patches
Oleg Bartunov <oleg@sai.msu.su> writes:
> On Tue, 9 Oct 2007, Andrew Dunstan wrote:
>> Are we in beta or not? To me, beta means nothing but bug fixes go in,
>> period. No ifs, no buts, no maybes, no exceptions. And that should
>> definitely go for contrib as well. We need a bit of self-discipline around
>> here.

> the original intention was to have text search companion with code examples
> for dictionaries and parser API and ability to keep them in workable
> condition, instead of having them in SGML documentation.

Yes.  I think that we can justify new contrib code here as a documentation
bug fix: right now, the examples in sections 12.9 and 12.10 are wrong
(obsolete), impossible to maintain (which is why they're obsolete;
they failed to track code changes), and not in a format that's directly
useful as a template for new code.

However, the dict_regex code is not ready for a number of reasons,
quite aside from the question of whether we want to add a dependency.
I concur with the idea of dropping it for now.

I'm a bit inclined to drop dict_roman as well, as it seems of
questionable real use, and code-example-wise it adds nothing over
dict_intdict.  That would leave us with two dictionary examples and one
parser example, and I think both of the dictionaries look useful enough
to be worth keeping.  (In particular I'd not vote for having only
dict_intdict since it does not illustrate how to use a config file
... and we surely don't want to encourage people to not do that
correctly.)

            regards, tom lane

pgsql-patches by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: Additional windows codepages
Next
From: Tom Lane
Date:
Subject: Re: Preliminary GSSAPI Patches