[
https://issues.apache.org/jira/browse/SOLR-15257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17344853#comment-17344853
]
David Smiley commented on SOLR-15257:
-------------------------------------
Couple additional points:
* The resulting Query ought to expose the DocSet (e.g. via a getter) to enable
optimizations (not yet written) in Solr. I'm thinking that
SolrIndexSearcher.getProcessedFilter (or a method it already calls) could
detect this and simply unwrap the Query to expose the DocSet. The Query could
perhaps implement DocSetProducer?
* Filter has a boolean "applyLazily" that effectively picks between an
iteration based algorithm, or TwoPhaseIterator. Within Solr, I believe it's
always "false" thus meaning _not_ TwoPhaseIterator. In this issue, we may want
to keep things as they are and leave performance implications to a separate
issue[1]. This only applies to BitDocSet, not the integer array one
[1] I can definitely imagine there is room for improvement here for Solr to
pick the right algorithm depending on the situation instead of always
iteration. The smaller the bitset, the more likely it ought to be iterator
based. If the Weight's BulkScorer is called (e.g. thus this is the *only*
query), then definitely iterator based, and could also skip applying deleted
docs (since DocSets are already filtered by them). I could also imagine a
hybrid iterator & TPI combo in which the approximation iterator looks at only
the "long" values, and the TPI part looks at a bitmask of an individual "long".
> Add DocSetQuery in lieu of DocSet.getTopFilter
> ----------------------------------------------
>
> Key: SOLR-15257
> URL: https://issues.apache.org/jira/browse/SOLR-15257
> Project: Solr
> Issue Type: Sub-task
> Components: search
> Reporter: David Smiley
> Priority: Major
>
> With Filter going away, we still need the ability to have a Query based on a
> DocSet. This currently happens via {{DocSet.getTopFilter}}. We could add a
> {{.makeQuery}} or just provide it.
> This issue depends on changes to DocSet's API to provide a DocIdSetIterator
> given a LeafReaderContext.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
