ACCUMULO-2369 Increase the sleep time and add some comments as to why

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

Branch: refs/heads/master
Commit: a9cd5d14e5675a6346bdce79bc5c670af88f784b
Parents: 1b41177
Author: Josh Elser <els...@apache.org>
Authored: Thu Feb 13 18:40:23 2014 -0500
Committer: Josh Elser <els...@apache.org>
Committed: Thu Feb 13 18:40:23 2014 -0500

----------------------------------------------------------------------
 .../vfs/AccumuloReloadingVFSClassLoaderTest.java            | 9 +++++++--
 .../start/classloader/vfs/providers/VfsClassLoaderTest.java | 3 +++
 2 files changed, 10 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/a9cd5d14/start/src/test/java/org/apache/accumulo/start/classloader/vfs/AccumuloReloadingVFSClassLoaderTest.java
----------------------------------------------------------------------
diff --git 
a/start/src/test/java/org/apache/accumulo/start/classloader/vfs/AccumuloReloadingVFSClassLoaderTest.java
 
b/start/src/test/java/org/apache/accumulo/start/classloader/vfs/AccumuloReloadingVFSClassLoaderTest.java
index 489dbb2..96ee517 100644
--- 
a/start/src/test/java/org/apache/accumulo/start/classloader/vfs/AccumuloReloadingVFSClassLoaderTest.java
+++ 
b/start/src/test/java/org/apache/accumulo/start/classloader/vfs/AccumuloReloadingVFSClassLoaderTest.java
@@ -103,11 +103,15 @@ public class AccumuloReloadingVFSClassLoaderTest {
     
     new File(folder1.getRoot(), "HelloWorld.jar").delete();
 
+    // VFS-487 significantly wait to avoid failure
+    Thread.sleep(7000);
+
     // Update the class
     FileUtils.copyURLToFile(this.getClass().getResource("/HelloWorld.jar"), 
folder1.newFile("HelloWorld2.jar"));
 
     //Wait for the monitor to notice
-    Thread.sleep(2000);
+    // VFS-487 significantly wait to avoid failure
+    Thread.sleep(7000);
     
     Class<?> clazz2 = arvcl.getClassLoader().loadClass("test.HelloWorld");
     Object o2 = clazz2.newInstance();
@@ -156,7 +160,8 @@ public class AccumuloReloadingVFSClassLoaderTest {
     FileUtils.copyURLToFile(this.getClass().getResource("/HelloWorld2.jar"), 
folder1.newFile("HelloWorld.jar"));
     
     // Wait for the monitor to notice
-    Thread.sleep(2000);
+    // VFS-487 significantly wait to avoid failure
+    Thread.sleep(7000);
     
 
     Class<?> clazz2 = arvcl.getClassLoader().loadClass("test.HelloWorld");

http://git-wip-us.apache.org/repos/asf/accumulo/blob/a9cd5d14/start/src/test/java/org/apache/accumulo/start/classloader/vfs/providers/VfsClassLoaderTest.java
----------------------------------------------------------------------
diff --git 
a/start/src/test/java/org/apache/accumulo/start/classloader/vfs/providers/VfsClassLoaderTest.java
 
b/start/src/test/java/org/apache/accumulo/start/classloader/vfs/providers/VfsClassLoaderTest.java
index 4f0f0e9..c0a07cd 100644
--- 
a/start/src/test/java/org/apache/accumulo/start/classloader/vfs/providers/VfsClassLoaderTest.java
+++ 
b/start/src/test/java/org/apache/accumulo/start/classloader/vfs/providers/VfsClassLoaderTest.java
@@ -80,6 +80,7 @@ public class VfsClassLoaderTest extends AccumuloDFSBase {
     Path dst = new Path(TEST_DIR, "HelloWorld2.jar");
     this.hdfs.copyFromLocalFile(src, dst);
 
+    // VFS-487 significantly wait to avoid failure
     Thread.sleep(7000);
     Assert.assertTrue(listener.isFileCreated());
 
@@ -89,10 +90,12 @@ public class VfsClassLoaderTest extends AccumuloDFSBase {
     dst = new Path(TEST_DIR, "HelloWorld2.jar");
     this.hdfs.copyFromLocalFile(src, dst);
 
+    // VFS-487 significantly wait to avoid failure
     Thread.sleep(7000);
     Assert.assertTrue(listener.isFileChanged());
     
     this.hdfs.delete(dst, false);
+    // VFS-487 significantly wait to avoid failure
     Thread.sleep(7000);
     Assert.assertTrue(listener.isFileDeleted());
     

Reply via email to