[jira] Commented: (LUCENE-370) BooleanQuery assumes everything else implements skipTo

["paul.elschot (JIRA)" <ji...@apache.org>]

    [ http://issues.apache.org/jira/browse/LUCENE-370?page=comments#action_12332149 ] 

paul.elschot commented on LUCENE-370:
-------------------------------------

I think all scorers in the trunk implement skipTo() now, so this can be closed.

Regards,
Paul Elschot

> BooleanQuery assumes everything else implements skipTo
> ------------------------------------------------------
>
>          Key: LUCENE-370
>          URL: http://issues.apache.org/jira/browse/LUCENE-370
>      Project: Lucene - Java
>         Type: Bug
>   Components: Search
>     Versions: 1.4
>  Environment: Operating System: other
> Platform: All
>     Reporter: Yonik Seeley
>     Assignee: Lucene Developers

>
> skipTo seems to be optional functionality on the Scorer class (BooleanScorer
> doesn't implement it).  BooleanQuery.scorer() tests all subclauses using
> "instanceof BooleanQuery" to determine if it can use a ConjunctionScorer that
> requires skipTo functionality.
> This means that any other new Query/Scorer that don't implement skipTo will get
> into trouble when included in a BooleanQuery.
> If skipTo is really optional, then there should be some way of telling by the
> Scorer or the Query in a more generic manner.
> Some options:
> 1) have a "boolean Scorer.hasSkipTo()" method
> 2) have a "boolean Query.hasSkipTo()" method
> 3) remove Scorer.skipTo and have a "public interface ScorerSkipTo{boolean
> skipTo(int doc)}" that scorers may implement

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: java-dev-help@lucene.apache.org