rmuir commented on PR #11906: URL: https://github.com/apache/lucene/pull/11906#issuecomment-1308119525
current test still doesn't fail. checkIndex just calls nextDoc() on low-level vectors but we may need to invoke skipping to find the issue. That's my theory at least. one thing missing is that, other than checking for vector's lengths and other super-basic stuff, we don't have any kind of "logical checksums" to ensure all the decoded data is really correct. For things like postings we have these (the index statistics: docFreq, termFreq, totalTermFreq, sumDocFreq, etc). That's going to limit how well we can find issues with tests. -- 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
