timeout on 100 sec

apache · Dec 6, 2023 · 4df00b9 · 4df00b9
1 parent 58c0484
commit 4df00b9
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/test/java/org/apache/sysds/test/AutomatedTestBase.java b/src/test/java/org/apache/sysds/test/AutomatedTestBase.java
@@ -125,11 +125,14 @@ public abstract class AutomatedTestBase {
 	public static final double GPU_TOLERANCE = 1e-9;
 
 	// ms wait time 
-	public static final int FED_WORKER_WAIT = 1000; 
+	public static final int FED_WORKER_WAIT = 1500; 
 	public static final int FED_MONITOR_WAIT = 10000; 
 	public static final int FED_WORKER_WAIT_S = 50;
 
 
+	// The timeout for a test to fail. all tests must execute in less than this time.
+	public static final int TEST_TIMEOUT = 100;
+
 	// With OpenJDK 8u242 on Windows, the new changes in JDK are not allowing
 	// to set the native library paths internally thus breaking the code.
 	// That is why, these static assignments to java.library.path and hadoop.home.dir
@@ -1442,13 +1445,12 @@ protected ByteArrayOutputStream runTest(boolean newWay, boolean exceptionExpecte
 		try{
 			return executor.submit(() -> 
 				runTestWithTimeout(newWay,exceptionExpected,expectedException,errMessage, maxSparkInst))//
-				.get(1000, TimeUnit.SECONDS);
+				.get(TEST_TIMEOUT, TimeUnit.SECONDS);
 		}
 		catch(TimeoutException e){
 			throw new RuntimeException("Our tests should run faster than 1000 sec each",e);
 		}
 		catch(Exception e){
-			fail(e.getMessage());
 			throw new RuntimeException(e);
 		}
 		finally{