thecoop opened a new pull request, #15044: URL: https://github.com/apache/lucene/pull/15044
Remove some assertions added by #14844 It turns out that merge instances can be created concurrently to getting byte/float values. This technically breaks the model of ReadAdvice used by MMapDirectory, as whilst a merge instance is open, any other accesses to values from another `KnnVectorsReader` will use the incorrect ReadAdvice. Whilst not bad, this may cause unexpected performance issues. This PR removes these assertions for now, pending further investigation -- 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
