gsmiller commented on code in PR #12194:
URL: https://github.com/apache/lucene/pull/12194#discussion_r1131775078


##########
lucene/core/src/java/org/apache/lucene/codecs/lucene90/Lucene90PostingsReader.java:
##########
@@ -479,6 +481,31 @@ private void refillDocs() throws IOException {
       assert docBuffer[BLOCK_SIZE] == NO_MORE_DOCS;
     }
 
+    @Override
+    public int peekNextNonMatchingDocID() throws IOException {

Review Comment:
   This won't be straight forward, but we have a "special" encoding whenever a 
block is "completely dense" (i.e., has a run of 128 consecutive docs). See 
`PForUtil#decodeAndPrefixSum` for details. I wonder if we can leverage that 
here? In extreme cases, we may have multiple blocks that are all consecutive. 
Could we "peek ahead" and discover that?



-- 
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

Reply via email to