Tom Lane wrote:
> Bruce Momjian <bruce(at)momjian(dot)us> writes:
> > Tom Lane wrote:
> >> One possible real solution would be to tweak the dictionary APIs so
> >> that the dictionaries can find out whether this is the first load during
> >> a session, or a reload, and emit notices only in the first case.
> 
> > Yea, that would work too.  Or just throw an error for a stop word in the
> > file and then you never get a reload (use "*" instead).
> 
> Hm, that's a thought --- it'd be a way to solve the problem without an
> API change for dictionaries, which is something to avoid at this late
> stage of the 8.3 cycle.  Come to think of it, does the ts_cache stuff
> work properly when an error is thrown in dictionary load (ie, is the
> cache entry left in a sane state)?

I have developed the attached patch which uses "?" to mark stop words in
the thesaurus file.  ("*" was already in use in the file.)  I updated
the docs to use "?", which makes the documentation clearer too.

The patch also reenables testing of stop words in the thesuarus file.

FYI, there is no longer a NOTICE for stop words in the thesaurus file; 
it throws an error now, and says to use "?" instead.

-- 
  Bruce Momjian  <bruce(at)momjian(dot)us>        http://momjian.us
  EnterpriseDB                             http://postgres.enterprisedb.com

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

In response to

• Re: [COMMITTERS] pgsql: Temporarily modify tsearch regression tests to suppress notice at 2007-09-29 14:50:34 from Tom Lane

Responses

• Re: Fix for stop words in thesaurus file at 2007-11-09 09:16:54 from Simon Riggs
• Re: Fix for stop words in thesaurus file at 2007-11-10 15:40:34 from Bruce Momjian

pgsql-hackers by date

Next:	From: Bruce Momjian	Date: 2007-11-09 02:50:45
	Subject: Re: [HACKERS] Connection Pools and DISCARD ALL
Previous:	From: Andrew Dunstan	Date: 2007-11-09 02:30:48
	Subject: Re: How to determine if psql returns ZERO to the "shell" and finished successfully?

pgsql-committers by date

Next:	From: Simon Riggs	Date: 2007-11-09 08:35:17
	Subject: Re: pgsql: If an index depends on no columns of itstable, give it a
Previous:	From: Bruce Momjian	Date: 2007-11-09 01:32:22
	Subject: pgsql: In tsearch code, remove !(A && B) via restructuring, for clarity

pgsql-patches by date

Next:	From: Bruce Momjian	Date: 2007-11-09 02:50:45
	Subject: Re: [HACKERS] Connection Pools and DISCARD ALL
Previous:	From: Jan Urbański	Date: 2007-11-09 01:22:34
	Subject: a tsearch2 (8.2.4) dictionary that only filters out stopwords