Done (PriorityQueue-2.patch) Shai
On Dec 12, 2007 1:46 PM, Michael McCandless <[EMAIL PROTECTED]> wrote: > > I think it's fine to include those changes with this issue. > > Mike > > Shai Erera wrote: > > > Hi > > > > I created https://issues.apache.org/jira/browse/LUCENE-1089 and > > added a > > patch. > > I noticed that we can replace the calls to insert() with > > insertWithOverflow() in several other places, like > > QualityQueriesFinder, > > FuzzyQuery and TopFieldDocCollector. I wasn't sure if that should > > be handled > > as part of this issue, or a different one. > > > > On Dec 11, 2007 8:32 PM, Yonik Seeley <[EMAIL PROTECTED]> wrote: > > > >> On Dec 11, 2007 1:21 PM, Timo Nentwig <[EMAIL PROTECTED]> wrote: > >>> On Tuesday 11 December 2007 14:32:12 Shai Erera wrote: > >>>> For (1) - I can't explain it but I've run into documents with > >>>> 0.0fscores. > >>>> For (2) - this is a simple logic - if the lowest score in the > >>>> queue is > >> 'x' > >>>> and you want to top docs only, then there's no point in > >>>> attempting to > >>>> insert a document with score lower than 'x' (it will not be added). > >>> > >>> Sure. I didn't notice that score is passed as parameter and was > >> surprised that > >>> subsequent calls to collect() are supposed to be guaranteed to > >>> have a > >> lower > >>> score. > >> > >> One is not guaranteed this... collect() generally goes in docid > >> order, > >> and scores are unordered. > >> > >> If you are only gathering the top 10 docs by score, you can compare > >> the current score to the lowest of the top 10 you currently have to > >> determine if you should bother inserting into the queue. > >> > >> -Yonik > >> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: [EMAIL PROTECTED] > >> For additional commands, e-mail: [EMAIL PROTECTED] > >> > >> > > > > > > -- > > Regards, > > > > Shai Erera > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- Regards, Shai Erera