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

Adrien Grand commented on LUCENE-6022:
--------------------------------------

I agree this thing could be merged with FilteredDocIdSet. The only additional 
thing it has is the optimization when deleted docs are a bit set but I'm 
wondering if it really helps in practice given that we try to merge more 
agressively segments that have lots of deleted documents.

> DocValuesDocIdSet: check deleted docs before doc values
> -------------------------------------------------------
>
>                 Key: LUCENE-6022
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6022
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Adrien Grand
>            Assignee: Adrien Grand
>            Priority: Trivial
>             Fix For: 5.0
>
>         Attachments: LUCENE-6022.patch
>
>
> When live documents are not null, DocValuesDocIdSet checks if doc values 
> match the document before the live docs. Given that checking if doc values 
> match could involve a heavy computation (eg. geo distance) and that the 
> default codec has live docs in memory but doc values on disk, I think it 
> makes more sense to check live docs first?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to