jpountz commented on PR #12549:
URL: https://github.com/apache/lucene/pull/12549#issuecomment-1777530886

   I reverted as this causes a deadlock in TestStressIndexing:
   
   ```
   
"TEST-TestStressIndexing.testStressIndexAndSearching-seed#[D4B60FA81EB58FF3]" 
#42 [282516] prio=5 os_prio=0 cpu=314.69ms elapsed=8.44s tid=0x00007f9144078860 
nid=282516 in Object.wait()  [0x00007f9195122000]
      java.lang.Thread.State: TIMED_WAITING (on object monitor)
        at java.lang.Object.wait0(java.base@21/Native Method)
        - waiting on <0x00000006207504f0> (a 
org.apache.lucene.index.IndexWriter)
        at java.lang.Object.wait(java.base@21/Object.java:366)
        at org.apache.lucene.index.IndexWriter.doWait(IndexWriter.java:5419)
        - locked <0x00000006207504f0> (a org.apache.lucene.index.IndexWriter)
        at 
org.apache.lucene.index.IndexWriter.abortMerges(IndexWriter.java:2723)
        - locked <0x00000006207504f0> (a org.apache.lucene.index.IndexWriter)
        at 
org.apache.lucene.index.IndexWriter.rollbackInternalNoCommit(IndexWriter.java:2471)
        - locked <0x00000006207504f0> (a org.apache.lucene.index.IndexWriter)
        at 
org.apache.lucene.index.IndexWriter.rollbackInternal(IndexWriter.java:2451)
        - locked <0x0000000620750a88> (a java.lang.Object)
        at org.apache.lucene.index.IndexWriter.shutdown(IndexWriter.java:1336)
        at org.apache.lucene.index.IndexWriter.close(IndexWriter.java:1364)
        at 
org.apache.lucene.tests.store.MockDirectoryWrapper.close(MockDirectoryWrapper.java:973)
        - locked <0x000000062702c9f8> (a 
org.apache.lucene.tests.store.MockDirectoryWrapper)
        at 
org.apache.lucene.index.TestStressIndexing.testStressIndexAndSearching(TestStressIndexing.java:171)
        at 
java.lang.invoke.LambdaForm$DMH/0x00007f919810d000.invokeVirtual(java.base@21/LambdaForm$DMH)
        at 
java.lang.invoke.LambdaForm$MH/0x00007f919810d400.invoke(java.base@21/LambdaForm$MH)
        at 
java.lang.invoke.Invokers$Holder.invokeExact_MT(java.base@21/Invokers$Holder)
        at 
jdk.internal.reflect.DirectMethodHandleAccessor.invokeImpl(java.base@21/DirectMethodHandleAccessor.java:153)
        at 
jdk.internal.reflect.DirectMethodHandleAccessor.invoke(java.base@21/DirectMethodHandleAccessor.java:103)
        at java.lang.reflect.Method.invoke(java.base@21/Method.java:580)
        at 
com.carrotsearch.randomizedtesting.RandomizedRunner.invoke(RandomizedRunner.java:1758)
        at 
com.carrotsearch.randomizedtesting.RandomizedRunner$8.evaluate(RandomizedRunner.java:946)
        at 
com.carrotsearch.randomizedtesting.RandomizedRunner$9.evaluate(RandomizedRunner.java:982)
        at 
com.carrotsearch.randomizedtesting.RandomizedRunner$10.evaluate(RandomizedRunner.java:996)
        at 
org.apache.lucene.tests.util.TestRuleSetupTeardownChained$1.evaluate(TestRuleSetupTeardownChained.java:48)
        at 
org.apache.lucene.tests.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:43)
        at 
org.apache.lucene.tests.util.TestRuleThreadAndTestName$1.evaluate(TestRuleThreadAndTestName.java:45)
        at 
org.apache.lucene.tests.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:60)
        at 
org.apache.lucene.tests.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:44)
        at org.junit.rules.RunRules.evaluate(RunRules.java:20)
        at 
com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
        at 
com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:390)
        at 
com.carrotsearch.randomizedtesting.ThreadLeakControl.forkTimeoutingTask(ThreadLeakControl.java:843)
        at 
com.carrotsearch.randomizedtesting.ThreadLeakControl$3.evaluate(ThreadLeakControl.java:490)
        at 
com.carrotsearch.randomizedtesting.RandomizedRunner.runSingleTest(RandomizedRunner.java:955)
        at 
com.carrotsearch.randomizedtesting.RandomizedRunner$5.evaluate(RandomizedRunner.java:840)
        at 
com.carrotsearch.randomizedtesting.RandomizedRunner$6.evaluate(RandomizedRunner.java:891)
        at 
com.carrotsearch.randomizedtesting.RandomizedRunner$7.evaluate(RandomizedRunner.java:902)
        at 
org.apache.lucene.tests.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:43)
        at 
com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
        at 
org.apache.lucene.tests.util.TestRuleStoreClassName$1.evaluate(TestRuleStoreClassName.java:38)
        at 
com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:40)
        at 
com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:40)
        at 
com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
        at 
com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
        at 
org.apache.lucene.tests.util.TestRuleAssertionsRequired$1.evaluate(TestRuleAssertionsRequired.java:53)
        at 
org.apache.lucene.tests.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:43)
        at 
org.apache.lucene.tests.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:44)
        at 
org.apache.lucene.tests.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:60)
        at 
org.apache.lucene.tests.util.TestRuleIgnoreTestSuites$1.evaluate(TestRuleIgnoreTestSuites.java:47)
        at org.junit.rules.RunRules.evaluate(RunRules.java:20)
        at 
com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
        at 
com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:390)
        at 
com.carrotsearch.randomizedtesting.ThreadLeakControl.lambda$forkTimeoutingTask$0(ThreadLeakControl.java:850)
        at 
com.carrotsearch.randomizedtesting.ThreadLeakControl$$Lambda/0x00007f919808ed60.run(Unknown
 Source)
        at java.lang.Thread.runWith(java.base@21/Thread.java:1596)
        at java.lang.Thread.run(java.base@21/Thread.java:1583)
   
   "Lucene Merge Thread #0" #53 [282533] daemon prio=5 os_prio=0 cpu=1.59ms 
elapsed=7.81s tid=0x00007f910815ee70 nid=282533 waiting for monitor entry  
[0x00007f9195022000]
      java.lang.Thread.State: BLOCKED (on object monitor)
        at 
org.apache.lucene.tests.store.MockDirectoryWrapper.createOutput(MockDirectoryWrapper.java:692)
        - waiting to lock <0x000000062702c9f8> (a 
org.apache.lucene.tests.store.MockDirectoryWrapper)
        at 
org.apache.lucene.store.LockValidatingDirectoryWrapper.createOutput(LockValidatingDirectoryWrapper.java:43)
        at 
org.apache.lucene.index.ConcurrentMergeScheduler$2.createOutput(ConcurrentMergeScheduler.java:290)
        at 
org.apache.lucene.store.TrackingDirectoryWrapper.createOutput(TrackingDirectoryWrapper.java:41)
        at 
org.apache.lucene.codecs.lucene90.compressing.Lucene90CompressingStoredFieldsWriter.<init>(Lucene90CompressingStoredFieldsWriter.java:130)
        at 
org.apache.lucene.codecs.lucene90.compressing.Lucene90CompressingStoredFieldsFormat.fieldsWriter(Lucene90CompressingStoredFieldsFormat.java:140)
        at 
org.apache.lucene.codecs.lucene90.Lucene90StoredFieldsFormat.fieldsWriter(Lucene90StoredFieldsFormat.java:154)
        at 
org.apache.lucene.index.SegmentMerger.mergeFields(SegmentMerger.java:233)
        at 
org.apache.lucene.index.SegmentMerger$$Lambda/0x00007f91981801f8.merge(Unknown 
Source)
        at 
org.apache.lucene.index.SegmentMerger.mergeWithLogging(SegmentMerger.java:273)
        at org.apache.lucene.index.SegmentMerger.merge(SegmentMerger.java:110)
        at 
org.apache.lucene.index.IndexWriter.mergeMiddle(IndexWriter.java:5172)
        at org.apache.lucene.index.IndexWriter.merge(IndexWriter.java:4705)
        at 
org.apache.lucene.index.IndexWriter$IndexWriterMergeSource.merge(IndexWriter.java:6460)
        at 
org.apache.lucene.index.ConcurrentMergeScheduler.doMerge(ConcurrentMergeScheduler.java:639)
        at 
org.apache.lucene.index.ConcurrentMergeScheduler$MergeThread.run(ConcurrentMergeScheduler.java:700)
   ```
   
   This seems to be more a problem with `MockDirectoryWrapper` than with the 
change to me, though I'm unsure how to fix MDW.


-- 
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

Reply via email to