Group: pgsql.hackers


Subject: Still a NOTICE in dict_thesaurus.c
From: tgl@sss.pgh.pa.us (Tom Lane)
Date: 11/27/2007 5:26:36 PM
Wasn't this patch http://archives.postgresql.org/pgsql-committers/2007-11/msg00170.php 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? http://www.postgresql.org/docs/faq

Subject: Still a NOTICE in dict_thesaurus.c
From: tgl@sss.pgh.pa.us (Tom Lane)
Date: 11/27/2007 6:15:28 PM
Bruce Momjian <bruce@momjian.us> writes: > 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. Why would you do that? There is not any corresponding idea of leaving holes in the pattern. AFAICS the only change needed is NOTICE->ERROR. regards, tom lane ---------------------------(end of broadcast)--------------------------- TIP 9: In versions below 8.0, the planner will ignore your desire to choose an index scan if your joining column's datatypes do not match

Subject: Still a NOTICE in dict_thesaurus.c
From: tgl@sss.pgh.pa.us (Tom Lane)
Date: 11/28/2007 10:06:16 AM
Alvaro Herrera <alvherre@alvh.no-ip.org> writes: > FYI, at this time only the German and French translations have been > updated almost fully, and none of them is yet 100% in the backend > translation. I think it's appropriate to mark this string for > translation at this time (i.e. turn it into an ereport call). Yeah, I saw a number of elogs that should be ereports in the tsearch code during some idle looking yesterday. I'm planning to go through it a bit more systematically today and fix what needs fixing. regards, tom lane ---------------------------(end of broadcast)--------------------------- TIP 9: In versions below 8.0, the planner will ignore your desire to choose an index scan if your joining column's datatypes do not match