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

Jason Rutherglen commented on LUCENE-1476:
------------------------------------------

{quote} 
Should we deprecate isDeleted? And also deprecate document()
checking whether a document is deleted (ie switch to a new API that
returns document w/o checking deletions). 
{quote}

Deprecating isDeleted might be good. Would we need the read only
readers? We can still offer get method access to the deleted docs if
it's a bit set by creating an abstract class DocIdBitSet that
BitVector, java.util.BitSet, and OpenBItSet implement. This can
happen by casting IR.getDeletedDocs to DocIdBItSet. 

{quote} 
Or perhaps move isDeleted to a new API that makes it clear
that there is a performance cost to random-access of deletions? (And
the first time it's called, it materializes a full bit vector).
{quote}

Seems best to keep deleted docs access to the DocIdSet and DocIdBitSet
and not have IR.isDeleted.

> BitVector implement DocIdSet, IndexReader returns DocIdSet deleted docs
> -----------------------------------------------------------------------
>
>                 Key: LUCENE-1476
>                 URL: https://issues.apache.org/jira/browse/LUCENE-1476
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: Index
>    Affects Versions: 2.4
>            Reporter: Jason Rutherglen
>            Priority: Trivial
>         Attachments: LUCENE-1476.patch, LUCENE-1476.patch, LUCENE-1476.patch, 
> quasi_iterator_deletions.diff, quasi_iterator_deletions_r2.diff
>
>   Original Estimate: 12h
>  Remaining Estimate: 12h
>
> Update BitVector to implement DocIdSet.  Expose deleted docs DocIdSet from 
> IndexReader.

-- 
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: java-dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: java-dev-h...@lucene.apache.org

Reply via email to