[GitHub] [lucene] gsmiller commented on issue #11547: IntersectIterators is not necessary under matchAll case in Facet [LUCENE-10511]

[GitBox <gi...@apache.org>]

gsmiller commented on issue #11547:
URL: https://github.com/apache/lucene/issues/11547#issuecomment-1238537126

   Interesting observation @LuXugang. We could only "know" ahead of time that multiple iterators are "the same" though when they're effectively "all" docs in a segment right? I can't really think of a way to generalize beyond that, but maybe I'm overlooking something? I'd be +1 to seeing if we can add some intelligence to `ConjunctionUtils` to handle the case where one of the iterators covers all docs in a segment though. Sounds like what you had in mind?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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