GEODE-1821: Adjust timing to wait for async thread

This closes #279


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

Branch: refs/heads/feature/GEODE-2017
Commit: dd2df31b42cdfdb2728828ff5640b897d480d819
Parents: 3cf725f
Author: Ken Howe <[email protected]>
Authored: Wed Nov 2 11:14:13 2016 -0700
Committer: Udo Kohlmeyer <[email protected]>
Committed: Tue Nov 8 05:39:39 2016 +1100

----------------------------------------------------------------------
 .../PersistentColocatedPartitionedRegionDUnitTest.java      | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/dd2df31b/geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentColocatedPartitionedRegionDUnitTest.java
----------------------------------------------------------------------
diff --git 
a/geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentColocatedPartitionedRegionDUnitTest.java
 
b/geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentColocatedPartitionedRegionDUnitTest.java
index 3f2ab70..a044a61 100644
--- 
a/geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentColocatedPartitionedRegionDUnitTest.java
+++ 
b/geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentColocatedPartitionedRegionDUnitTest.java
@@ -93,7 +93,7 @@ public class PersistentColocatedPartitionedRegionDUnitTest
   private static final String PATTERN_FOR_MISSING_CHILD_LOG =
       "(?s)Persistent data recovery for region .*is prevented by offline 
colocated region.*";
   private static final int NUM_BUCKETS = 15;
-  private static final int MAX_WAIT = 30 * 1000;
+  private static final int MAX_WAIT = 60 * 1000;
   private static final int DEFAULT_NUM_EXPECTED_LOG_MESSAGES = 1;
   private static int numExpectedLogMessages = 
DEFAULT_NUM_EXPECTED_LOG_MESSAGES;
   private static int numChildPRs = 1;
@@ -1188,6 +1188,13 @@ public class 
PersistentColocatedPartitionedRegionDUnitTest
     CHILD_REGION_RESTART_ORDER.add(new Object[] {"Gen2_C2_2", "Gen1_C2", 0});
   }
 
+  /**
+   * This thread starts up multiple colocated child regions in the sequence 
defined by
+   * {@link #CHILD_REGION_RESTART_ORDER}. The complete startup sequence, which 
includes timed
+   * periods waiting for log messages, takes at least 28 secs. Tests waiting 
for this
+   * {@link SerializableCallable} to complete must have sufficient overhead in 
the wait for runtime
+   * variations that exceed the minimum time to complete.
+   */
   private SerializableCallable createPRsSequencedColocationTreeCreationThread =
       new SerializableCallable("createPRsSequencedColocationTreeCreation") {
         Appender mockAppender;

Reply via email to