benwtrent commented on PR #13475:
URL: https://github.com/apache/lucene/pull/13475#issuecomment-2158456017
Making `norms` & `terms` merge synchronously (not forking to the intra-merge
pool), makes the assertions go away, but then (surprise surprise), another test
fails 🤦
```
gradlew :lucene:core:test --tests
"org.apache.lucene.index.TestIndexWriterForceMerge.testMergePerField"
-Ptests.jvms=5 "-Ptests.jvmargs=-XX:TieredStopAtLevel=1 -XX:+UseParallelGC
-XX:ActiveProcessorCount=1" -Ptests.seed=FF93459708A1FFD4 -Ptests.gui=false
-Ptests.file.encoding=US-ASCII -Ptests.vectorsize=128
-Ptests.forceintegervectors=true
```
```
TestIndexWriterForceMerge > testMergePerField FAILED
java.lang.AssertionError: broken barrier
at
org.apache.lucene.index.TestIndexWriterForceMerge$BlockingOnMergePostingsFormat$1.merge(TestIndexWriterForceMerge.java:401)
at
org.apache.lucene.codecs.perfield.PerFieldPostingsFormat$FieldsWriter.merge(PerFieldPostingsFormat.java:204)
at
org.apache.lucene.index.SegmentMerger.mergeTerms(SegmentMerger.java:230)
at
org.apache.lucene.index.SegmentMerger.mergeWithLogging(SegmentMerger.java:319)
at
org.apache.lucene.index.SegmentMerger.merge(SegmentMerger.java:144)
at
org.apache.lucene.index.IndexWriter.mergeMiddle(IndexWriter.java:5284)
at org.apache.lucene.index.IndexWriter.merge(IndexWriter.java:4752)
at
org.apache.lucene.index.IndexWriter$IndexWriterMergeSource.merge(IndexWriter.java:6573)
at
org.apache.lucene.index.ConcurrentMergeScheduler.doMerge(ConcurrentMergeScheduler.java:668)
at
org.apache.lucene.index.ConcurrentMergeScheduler$MergeThread.run(ConcurrentMergeScheduler.java:729)
Caused by:
java.util.concurrent.TimeoutException
at
java.base/java.util.concurrent.CyclicBarrier.dowait(CyclicBarrier.java:259)
at
java.base/java.util.concurrent.CyclicBarrier.await(CyclicBarrier.java:437)
at
org.apache.lucene.index.TestIndexWriterForceMerge$BlockingOnMergePostingsFormat$1.merge(TestIndexWriterForceMerge.java:399)
... 9 more
java.lang.AssertionError: broken barrier
at
org.apache.lucene.index.TestIndexWriterForceMerge$BlockingOnMergePostingsFormat$1.merge(TestIndexWriterForceMerge.java:401)
at
org.apache.lucene.codecs.perfield.PerFieldPostingsFormat$FieldsWriter.merge(PerFieldPostingsFormat.java:204)
at
org.apache.lucene.index.SegmentMerger.mergeTerms(SegmentMerger.java:230)
at
org.apache.lucene.index.SegmentMerger.mergeWithLogging(SegmentMerger.java:319)
at
org.apache.lucene.index.SegmentMerger.merge(SegmentMerger.java:144)
at
org.apache.lucene.index.IndexWriter.mergeMiddle(IndexWriter.java:5284)
at org.apache.lucene.index.IndexWriter.merge(IndexWriter.java:4752)
at
org.apache.lucene.index.IndexWriter$IndexWriterMergeSource.merge(IndexWriter.java:6573)
at
org.apache.lucene.index.ConcurrentMergeScheduler.doMerge(ConcurrentMergeScheduler.java:668)
at
org.apache.lucene.index.ConcurrentMergeScheduler$MergeThread.run(ConcurrentMergeScheduler.java:729)
Caused by:
java.util.concurrent.TimeoutException
at
java.base/java.util.concurrent.CyclicBarrier.dowait(CyclicBarrier.java:259)
at
java.base/java.util.concurrent.CyclicBarrier.await(CyclicBarrier.java:437)
at
org.apache.lucene.index.TestIndexWriterForceMerge$BlockingOnMergePostingsFormat$1.merge(TestIndexWriterForceMerge.java:399)
... 9 more
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
