[ 
https://issues.apache.org/jira/browse/LUCENE-1296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12646954#action_12646954
 ] 

Paul Elschot commented on LUCENE-1296:
--------------------------------------

{quote}Also, why not put your logic to sometimes choose a SortedVIntList impl 
intoCachingWrapperFilter.docIdSetToCache by default?{quote}

The main reason is that there are circumstances under which skipTo() is faster 
on an OpenBitSet than on a SortedVIntList. OpenBitSet allows random access, so 
it can start the skip from any point, but SortedVIntList can only start the 
skip from its current position.
OTOH SortedVIntList does have the advantage of being smaller when the set is 
sparse, and this may bring garbage collection advantages.
In all, not completely convincing either way.



> Allow use of compact DocIdSet in CachingWrapperFilter
> -----------------------------------------------------
>
>                 Key: LUCENE-1296
>                 URL: https://issues.apache.org/jira/browse/LUCENE-1296
>             Project: Lucene - Java
>          Issue Type: New Feature
>          Components: Search
>            Reporter: Paul Elschot
>            Assignee: Michael McCandless
>            Priority: Minor
>             Fix For: 2.9
>
>         Attachments: cachedFilter20080529.patch, cachedFilter20080605.patch
>
>
> Extends CachingWrapperFilter with a protected method to determine the 
> DocIdSet to be cached.

-- 
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]

Reply via email to