I'm just a user and haven't tried Paul's DisjunctionScorer yet, but I must say this sounds like a great idea. The ability to specialize combineScores() is a significant advantage. I'd suggest adding DisjunctionMaxScorer in addition to DisjunctionSumScorer and DisjunctionSumCoordScorer in the default distribution, and adding a variant to MultiFieldQueryParser that uses these to expand the query differently (expand the innermost clauses to search the multiple fields with Max rather than duplicating the original query across each field; I have code to do that for my MaxDisjunctionQuery which could be trivially changed to use DisjunctionMaxScorer if this was desired). I expect the variant MultiFieldQueryParser would be what most users want most of the time (e.g., it does a much better job searching for "red elephants" across fields "title" and "description").
Chuck > -----Original Message----- > From: Doug Cutting [mailto:[EMAIL PROTECTED] > Sent: Friday, December 10, 2004 12:36 PM > To: Lucene Developers List > Subject: Re: Boolean Scorer > > Christoph Goller wrote: > > I think we should change BooleanScorer. An easy way would be to sort > the > > bucket > > list before it is used. Do you think that would affect performance > > dramatically? > > I think it would make it slower. > > > Otherwise we should reimplement BooleanScorer. I haven't looked into > the > > DisjunctionScorer patch in Bugzilla yet. Maybe it's a good starting > point. > > I think we should incorporate Paul's code into CVS. This algorithm may > be slower in some cases, but it may also be faster in some cases. We > should add a static method to switch back to the old implementation, and > encourage folks to benchmark their code. If it proves no slower then we > could remove the old implementation altogether. > > What do others think? > > Paul's code is in: > > http://issues.apache.org/bugzilla/show_bug.cgi?id=31785 > > Has anyone tried this? > > Doug > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
