[ https://issues.apache.org/jira/browse/LUCENE-914?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12502186 ]
Yonik Seeley commented on LUCENE-914: ------------------------------------- ConstantScorer is implemented like: public boolean skipTo(int target) throws IOException { doc = bits.nextSetBit(target); // requires JDK 1.4 return doc >= 0; } The fix would be: public boolean skipTo(int target) throws IOException { doc = bits.nextSetBit( Math.max(target, doc+1) ); return doc >= 0; } But I'm really not sure I prefer that over just clarifying/restricting skipTo. > Scorer.skipTo(current) remains on current for some scorers > ---------------------------------------------------------- > > Key: LUCENE-914 > URL: https://issues.apache.org/jira/browse/LUCENE-914 > Project: Lucene - Java > Issue Type: Bug > Components: Search > Reporter: Doron Cohen > Priority: Minor > Attachments: lucene-914.patch > > > Background in http://www.nabble.com/scorer.skipTo%28%29-contr-tf3880986.html > It appears that several scorers do not strictly follow the spec of > Scorer.skipTo(n), and skip to current location remain in current location > whereas the spec says: "beyond current". > We should (probably) either relax the spec or fix the implementations. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]