You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by dw...@apache.org on 2026/08/25 19:57:40 UTC

(lucene) branch branch_10x updated: Correct the code not to rely on wall clock 1 second in waiter.join. (#16558)

This is an automated email from the ASF dual-hosted git repository.

dweiss pushed a commit to branch branch_10x
in repository https://gitbox.apache.org/repos/asf/lucene.git


The following commit(s) were added to refs/heads/branch_10x by this push:
     new 8fd4914a7e0 Correct the code not to rely on wall clock 1 second in waiter.join. (#16558)
8fd4914a7e0 is described below

commit 8fd4914a7e06716a684b64b699c039cda8647e12
Author: Dawid Weiss <da...@carrotsearch.com>
AuthorDate: Tue Aug 25 21:57:11 2026 +0200

    Correct the code not to rely on wall clock 1 second in waiter.join. (#16558)
---
 .../apache/lucene/search/TestControlledRealTimeReopenThread.java   | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/lucene/core/src/test/org/apache/lucene/search/TestControlledRealTimeReopenThread.java b/lucene/core/src/test/org/apache/lucene/search/TestControlledRealTimeReopenThread.java
index 2c40a4e7486..70085d2be1d 100644
--- a/lucene/core/src/test/org/apache/lucene/search/TestControlledRealTimeReopenThread.java
+++ b/lucene/core/src/test/org/apache/lucene/search/TestControlledRealTimeReopenThread.java
@@ -400,11 +400,8 @@ public class TestControlledRealTimeReopenThread extends ThreadedIndexingAndSearc
         };
     waiter.start();
     manager.maybeRefresh();
-    waiter.join(1000);
-    if (!finished.get()) {
-      waiter.interrupt();
-      fail("thread deadlocked on waitForGeneration");
-    }
+    waiter.join();
+    assertTrue("waiter thread terminated abnormally", finished.get());
     thread.close();
     thread.join();
     writer.close();