rmuir commented on PR #11738: URL: https://github.com/apache/lucene/pull/11738#issuecomment-1240897004
@gsmiller I think the question is, is it worth adding all those extra conditionals? I don't think the `DocIdSet#all` will really be that much faster in practice (I'm not even sure how often this optimization will hit for users anyway). Usually if a term is so dense that it is matching all the documents, then we aren't really reading actual postings anyway: we basically just read a single byte header that means "all ones" for the 128-doc FOR block and don't do any actual read/decoding. -- 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-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
