jpountz commented on PR #13585: URL: https://github.com/apache/lucene/pull/13585#issuecomment-2262631387
Things got a bit better later on (https://github.com/apache/lucene/pull/13585#issuecomment-2246112137), but your reading is correct that some queries get slower. This seems to especially affect queries that rely on performance of `BlockDocsEnum#nextDoc()` such as disjunction counts and queries sorted by field. This is because `BlockDocsEnum#nextDoc()` could previously completely ignore skip data, since it was stored separately, while it now has to skip over it by reading a vInt encoding the number of bytes for skip data and then calling `IndexInput#skipBytes` over this number of bytes. `CountOrHighHigh` runs are particularly noisy on my machine, apparently because my JVM makes different inlining decision on every run, I'm looking forward to nightly benchmarks getting published to get a better estimation of the slowdown. -- 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
