Tony-X commented on issue #12536: URL: https://github.com/apache/lucene/issues/12536#issuecomment-1706979791
Today when it skips, the skipper can tell us 1) the offset of the position block we should seek to 2) how many positions it needs to skip within this block. This is because post-skipping the first doc's position is not always at the start of a positions block (that's actually the most common case). Additionally, advancing to a specific doc that is not the start of a block means we need to skip those docs as well as their positions. In theory, if the skipper can tell us how many positions it has skipped that would work. This will require storing more information in the skip data than the current scheme. > Maybe make a PR adding a nice comment explaining why we really do need this int lastPosBlockOffset? Sure, I can enhance the documentation of `IntBlockTermState` -- 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
