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

Yonik Seeley commented on LUCENE-330:
-------------------------------------

There are multiple related issues here
 - introduction and use of SkipFilter, which returns DocNrSkipper instead of a 
BitSet.
 - changes to IndexSearcher.search(...,filter) to create and use a FilteredQuery

Without taking the full plunge into DocNrSkippers yet, the applicable patch to 
keep FilteredQuery from
matching filtered documents would be
http://issues.apache.org/jira/secure/attachment/12312490/FilteredQueryPatch1.txt

> There is a comment in some of your code that this doesn't work with 
> BooleanQuery though
The patch I referenced should work.  Paul is refering to BooleanScorer which 
doesn't implement skipTo, as opposed to BooleanScorer2, which does.

*If* the 1.4 BooleanScorer isn't used, then FilteredQueryPatch1.txt  can be 
made more efficient by using scorer.skipTo() in conjunction with 
bitset.nextSetBit() (per Paul's comments).


> [PATCH] Use filter bits for next() and skipTo() in FilteredQuery
> ----------------------------------------------------------------
>
>          Key: LUCENE-330
>          URL: http://issues.apache.org/jira/browse/LUCENE-330
>      Project: Lucene - Java
>         Type: Improvement
>   Components: Search
>     Versions: CVS Nightly - Specify date in submission
>  Environment: Operating System: other
> Platform: Other
>     Reporter: paul.elschot
>     Assignee: Lucene Developers
>     Priority: Minor
>  Attachments: FilteredQuery.java, FilteredQuery.java, FilteredQuery.java, 
> FilteredQuery.java, FilteredQueryPatch1.txt, IndexSearcherPatch2.txt, 
> SkipFilter.java, SkipFilter.java
>
> This improves performance of FilteredQuery by not calling score() 
> on documents that do not pass the filter. 
> This passes the current tests for FilteredQuery, but these tests 
> have not been adapted/extended.

-- 
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to