You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@twill.apache.org by ch...@apache.org on 2014/07/29 08:56:42 UTC

[3/7] git commit: Increase time limit for testing.

Increase time limit for testing.

Project: http://git-wip-us.apache.org/repos/asf/incubator-twill/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-twill/commit/da9e0665
Tree: http://git-wip-us.apache.org/repos/asf/incubator-twill/tree/da9e0665
Diff: http://git-wip-us.apache.org/repos/asf/incubator-twill/diff/da9e0665

Branch: refs/heads/feature/travis-build
Commit: da9e0665ae733814d69354499029d020ad104cb5
Parents: bc60724
Author: Terence Yim <te...@continuuity.com>
Authored: Mon Jul 28 22:18:04 2014 -0700
Committer: Terence Yim <te...@continuuity.com>
Committed: Mon Jul 28 22:18:04 2014 -0700

----------------------------------------------------------------------
 .../java/org/apache/twill/yarn/LogHandlerTestRun.java     |  2 +-
 .../java/org/apache/twill/yarn/ResourceReportTestRun.java | 10 +++++-----
 2 files changed, 6 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-twill/blob/da9e0665/twill-yarn/src/test/java/org/apache/twill/yarn/LogHandlerTestRun.java
----------------------------------------------------------------------
diff --git a/twill-yarn/src/test/java/org/apache/twill/yarn/LogHandlerTestRun.java b/twill-yarn/src/test/java/org/apache/twill/yarn/LogHandlerTestRun.java
index a566515..8a60174 100644
--- a/twill-yarn/src/test/java/org/apache/twill/yarn/LogHandlerTestRun.java
+++ b/twill-yarn/src/test/java/org/apache/twill/yarn/LogHandlerTestRun.java
@@ -69,7 +69,7 @@ public class LogHandlerTestRun extends BaseYarnTest {
                                        .start();
 
     Services.getCompletionFuture(controller).get();
-    Assert.assertTrue(latch.await(30, TimeUnit.SECONDS));
+    Assert.assertTrue(latch.await(100, TimeUnit.SECONDS));
 
     // Verify the log throwable
     Assert.assertEquals(1, throwables.size());

http://git-wip-us.apache.org/repos/asf/incubator-twill/blob/da9e0665/twill-yarn/src/test/java/org/apache/twill/yarn/ResourceReportTestRun.java
----------------------------------------------------------------------
diff --git a/twill-yarn/src/test/java/org/apache/twill/yarn/ResourceReportTestRun.java b/twill-yarn/src/test/java/org/apache/twill/yarn/ResourceReportTestRun.java
index 71bb607..e402042 100644
--- a/twill-yarn/src/test/java/org/apache/twill/yarn/ResourceReportTestRun.java
+++ b/twill-yarn/src/test/java/org/apache/twill/yarn/ResourceReportTestRun.java
@@ -155,10 +155,10 @@ public final class ResourceReportTestRun extends BaseYarnTest {
       }
     }, Threads.SAME_THREAD_EXECUTOR);
 
-    Assert.assertTrue(running.await(30, TimeUnit.SECONDS));
+    Assert.assertTrue(running.await(120, TimeUnit.SECONDS));
 
     Iterable<Discoverable> echoServices = controller.discoverService("echo");
-    Assert.assertTrue(YarnTestUtils.waitForSize(echoServices, 2, 60));
+    Assert.assertTrue(YarnTestUtils.waitForSize(echoServices, 2, 120));
     // check that we have 2 runnables.
     ResourceReport report = controller.getResourceReport();
     Assert.assertEquals(2, report.getRunnableResources("BuggyServer").size());
@@ -176,7 +176,7 @@ public final class ResourceReportTestRun extends BaseYarnTest {
 
     // takes some time for app master to find out the container completed...
     int count = 0;
-    while (count < 20) {
+    while (count < 100) {
       report = controller.getResourceReport();
       // check that we have 1 runnable, not 2.
       if (report.getRunnableResources("BuggyServer").size() == 1) {
@@ -188,7 +188,7 @@ public final class ResourceReportTestRun extends BaseYarnTest {
     }
     Assert.assertTrue("Still has 2 contains running after 20 seconds", count < 20);
 
-    controller.stop().get(30, TimeUnit.SECONDS);
+    controller.stop().get(100, TimeUnit.SECONDS);
     // Sleep a bit before exiting.
     TimeUnit.SECONDS.sleep(2);
   }
@@ -217,7 +217,7 @@ public final class ResourceReportTestRun extends BaseYarnTest {
 
     // wait for 3 echo servers to come up
     Iterable<Discoverable> echoServices = controller.discoverService("echo");
-    Assert.assertTrue(YarnTestUtils.waitForSize(echoServices, 3, 60));
+    Assert.assertTrue(YarnTestUtils.waitForSize(echoServices, 3, 120));
     ResourceReport report = controller.getResourceReport();
     // make sure resources for echo1 and echo2 are there
     Map<String, Collection<TwillRunResources>> usedResources = report.getResources();