azkaban-aplcache

Test stability: longer max wait (#1207) Additional stability …

6/12/2017 12:40:43 AM

even if test runner is slow (like supposedly on Travis sometimes)
This doesn't make the tests any slower than before, as long as they pass
Why would this help? Even locally the FlowRunner tests started failing when I put 10 instead of 100 max tries (10 ms wait per try). 1000 should make it pretty robust :)

For background, see:

PR #1179 (comment)
https://travis-ci.org/azkaban/azkaban/builds/240580566?utm_source=github_status&utm_medium=notification
PR #1201 (comment)
PR #1186

Juho Autio

Commit: 83697e9

Tree: 061dc86

Parents: df1dd2e

Changes

azkaban-exec-server/src/test/java/azkaban/execapp/FlowRunnerTestBase.java 2(+1 -1)

Details

azkaban-exec-server/src/test/java/azkaban/execapp/FlowRunnerTestBase.java 2(+1 -1)

diff --git a/azkaban-exec-server/src/test/java/azkaban/execapp/FlowRunnerTestBase.java b/azkaban-exec-server/src/test/java/azkaban/execapp/FlowRunnerTestBase.java
index 27775cb..d7d8d70 100644
--- a/azkaban-exec-server/src/test/java/azkaban/execapp/FlowRunnerTestBase.java
+++ b/azkaban-exec-server/src/test/java/azkaban/execapp/FlowRunnerTestBase.java
@@ -118,7 +118,7 @@ public class FlowRunnerTestBase {
   }
 
   private void waitForStatus(ExecutableNode node, Status status) {
-    for (int i = 0; i < 100; i++) {
+    for (int i = 0; i < 1000; i++) {
       if (node.getStatus() == status) {
         break;
       }