dnhatn opened a new pull request #182: URL: https://github.com/apache/lucene/pull/182
The newly added [assertion](https://github.com/apache/lucene/blob/50607e0fb9090a1321093aca4117b6560c511af0/lucene/core/src/java/org/apache/lucene/codecs/lucene90/compressing/Lucene90CompressingTermVectorsWriter.java#L898) in the bulk-merge logic doesn't always hold because we do not create a new instance of `Lucene90CompressingTermVectorsReader` for merges and that reader can be accessed in tests (as long as it happens on the same thread). This change clones a new term vectors reader for merges. The issue was uncovered by Elastic CI: https://elasticsearch-ci.elastic.co/job/apache+lucene-solr+nightly+main/361/consoleText. -- 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. 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
