You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hama.apache.org by ed...@apache.org on 2013/09/01 05:36:20 UTC

svn commit: r1519239 - /hama/trunk/core/src/test/java/org/apache/hama/bsp/sync/TestSyncServiceFactory.java

Author: edwardyoon
Date: Sun Sep  1 03:36:20 2013
New Revision: 1519239

URL: http://svn.apache.org/r1519239
Log:
HAMA-587: Fix unit test fails. Use just default ZK session timeout.

Modified:
    hama/trunk/core/src/test/java/org/apache/hama/bsp/sync/TestSyncServiceFactory.java

Modified: hama/trunk/core/src/test/java/org/apache/hama/bsp/sync/TestSyncServiceFactory.java
URL: http://svn.apache.org/viewvc/hama/trunk/core/src/test/java/org/apache/hama/bsp/sync/TestSyncServiceFactory.java?rev=1519239&r1=1519238&r2=1519239&view=diff
==============================================================================
--- hama/trunk/core/src/test/java/org/apache/hama/bsp/sync/TestSyncServiceFactory.java (original)
+++ hama/trunk/core/src/test/java/org/apache/hama/bsp/sync/TestSyncServiceFactory.java Sun Sep  1 03:36:20 2013
@@ -116,7 +116,6 @@ public class TestSyncServiceFactory exte
     conf.setInt(Constants.PEER_PORT, zkPort);
     conf.set(Constants.ZOOKEEPER_QUORUM, "localhost");
     conf.setInt(Constants.ZOOKEEPER_CLIENT_PORT, zkPort);
-    conf.set(Constants.ZOOKEEPER_SESSION_TIMEOUT, "12000");
     System.setProperty("user.dir", "/tmp");
     // given null, should return zookeeper
     final SyncServer syncServer = SyncServiceFactory.getSyncServer(conf);
@@ -147,8 +146,11 @@ public class TestSyncServiceFactory exte
         }
       }
     });
+    
+    Thread.sleep(3000);
 
     IntWritable data = new IntWritable(5);
+    
     syncClient.storeInformation(
         syncClient.constructKey(jobId, String.valueOf(1L), "test"), data, true,
         null);