Fix TestIndexWriterWithThreads#testIOExceptionDuringAbortWithThreadsOnlyOnce (apache#14424)

guojialiang92 · jpountz · commit 88c37a4b68d6 · 2025-04-24T22:13:14.000+02:00
* fix TestIndexWriterWithThreads#testIOExceptionDuringWriteSegmentWithThreadsOnlyOnce

Signed-off-by: guojialiang &lt;guojialiang.2012@bytedance.com&gt;

* fix TestIndexWriterWithThreads#testIOExceptionDuringWriteSegmentWithThreadsOnlyOnce

Signed-off-by: guojialiang &lt;guojialiang.2012@bytedance.com&gt;

* update

Signed-off-by: guojialiang &lt;guojialiang.2012@bytedance.com&gt;

---------

Signed-off-by: guojialiang &lt;guojialiang.2012@bytedance.com&gt;
diff --git a/lucene/core/src/test/org/apache/lucene/index/TestIndexWriterWithThreads.java b/lucene/core/src/test/org/apache/lucene/index/TestIndexWriterWithThreads.java
@@ -319,6 +319,8 @@ public void _testMultipleThreadsFailure(MockDirectoryWrapper.Failure failure) th
           IOException ioe) {
         writer.rollback();
         failure.clearDoFail();
+      } finally {
+        writer.close();
       }
       if (VERBOSE) {
         System.out.println("TEST: success=" + success);

Original file line number	Diff line number	Diff line change
`@@ -319,6 +319,8 @@ public void _testMultipleThreadsFailure(MockDirectoryWrapper.Failure failure) th`
`319`	`319`	`IOException ioe) {`
`320`	`320`	`writer.rollback();`
`321`	`321`	`failure.clearDoFail();`
	`322`	`+ } finally {`
	`323`	`+ writer.close();`
`322`	`324`	`}`
`323`	`325`	`if (VERBOSE) {`
`324`	`326`	`System.out.println("TEST: success=" + success);`