You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "David Smiley (Jira)" <ji...@apache.org> on 2020/02/15 05:42:00 UTC

[jira] [Commented] (SOLR-14256) Remove HashDocSet

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

David Smiley commented on SOLR-14256:
-------------------------------------

For better or worse, the code is on this PR for a related issue: https://github.com/apache/lucene-solr/pull/1257

> Remove HashDocSet
> -----------------
>
>                 Key: SOLR-14256
>                 URL: https://issues.apache.org/jira/browse/SOLR-14256
>             Project: Solr
>          Issue Type: Task
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: search
>            Reporter: David Smiley
>            Priority: Major
>
> This particular DocSet is only used in places where we need to convert SortedIntDocSet in particular to a DocSet that is fast for random access.  Once such a conversion happens, it's only used to test some docs for presence and it could be another interface.  DocSet has kind of a large-ish API surface area to implement.  Since we only need to test docs, we could use Bits interface (having only 2 methods) backed by an off-the-shelf primitive long hash set on our classpath.  Perhaps a new method on DocSet: getBits() or DocSetUtil.getBits(DocSet).
> In addition to removing complexity unto itself, this improvement is required by SOLR-14185 because it wants to be able to produce a DocIdSetIterator slice directly from the DocSet but HashDocSet can't do that without sorting first.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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