Skip site navigation (1) Skip section navigation (2)

Re: Still a NOTICE in dict_thesaurus.c

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Oleg Bartunov <oleg(at)sai(dot)msu(dot)su>, Teodor Sigaev <teodor(at)sigaev(dot)ru>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Still a NOTICE in dict_thesaurus.c
Date: 2007-11-27 23:13:35
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgsql-hackers
Thanks, working on this now.  Because the stop words can be any stop
word I didn't imagine it could be in the subsitution.  What stop word
will it use?  Anyway, coding it now to match the left-hand-side.


Tom Lane wrote:
> Wasn't this patch
> supposed to fix things so that we wouldn't be throwing NOTICEs out of
> tsearch dictionary init functions?  It seems to have gotten only
> one of the two elog(NOTICE) calls in there.  The other one is for a
> stopword in the replacement phrase.  Is there any objection to making
> that an ERROR too?
> Also, several of the elog's in this file ought to be ereport's since
> they are user-caused error conditions.  I think we are nominally past
> string freeze, but does anyone have an objection to changing them to
> ereport's?  If they don't get translated, the worst that will happen
> is that they come out in English, which is what's happening now.
> 			regards, tom lane
> ---------------------------(end of broadcast)---------------------------
> TIP 3: Have you checked our extensive FAQ?

  Bruce Momjian  <bruce(at)momjian(dot)us>

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

In response to


pgsql-hackers by date

Next:From: Tom LaneDate: 2007-11-27 23:15:28
Subject: Re: Still a NOTICE in dict_thesaurus.c
Previous:From: Joshua D. DrakeDate: 2007-11-27 23:08:40
Subject: Re: Quality and Performance

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group