You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ji...@apache.org on 2016/04/11 17:13:49 UTC

[20/24] incubator-geode git commit: GEODE-1062: New method to modify wait time for validateRegionSize

GEODE-1062: New method to modify wait time for validateRegionSize

Part of the refactoring work for GEODE-1062 led to a possible test failure
due to insufficient wait time for very slow tests.  The tests need to be fixed
but a quick workaround is to pass in the custom time of the original default time
of 240000


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

Branch: refs/heads/feature/GEODE-17-2
Commit: c23ec7a80afbf3c83e71de89581613dbb1b2a24b
Parents: c1cf90e
Author: Jason Huynh <hu...@gmail.com>
Authored: Fri Apr 8 11:35:06 2016 -0700
Committer: Jason Huynh <hu...@gmail.com>
Committed: Fri Apr 8 15:49:20 2016 -0700

----------------------------------------------------------------------
 .../com/gemstone/gemfire/internal/cache/wan/WANTestBase.java   | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/c23ec7a8/geode-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/WANTestBase.java
----------------------------------------------------------------------
diff --git a/geode-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/WANTestBase.java b/geode-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/WANTestBase.java
index 130f960..e93c9c2 100644
--- a/geode-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/WANTestBase.java
+++ b/geode-wan/src/test/java/com/gemstone/gemfire/internal/cache/wan/WANTestBase.java
@@ -3611,6 +3611,10 @@ public class WANTestBase extends DistributedTestCase{
   }
 
   public static void validateRegionSize(String regionName, final int regionSize) {
+    validateRegionSize(regionName, regionSize, 30000);
+  }
+
+  public static void validateRegionSize(String regionName, final int regionSize, long waitTime) {
     IgnoredException exp = IgnoredException.addIgnoredException(ForceReattemptException.class
         .getName());
     IgnoredException exp1 = IgnoredException.addIgnoredException(CacheClosedException.class
@@ -3633,7 +3637,7 @@ public class WANTestBase extends DistributedTestCase{
               + " present region keyset " + r.keySet();
         }
       };
-      Wait.waitForCriterion(wc, 30000, 500, true);
+      Wait.waitForCriterion(wc, waitTime, 500, true);
     } finally {
       exp.remove();
       exp1.remove();