Repository: incubator-geode
Updated Branches:
  refs/heads/develop ffd920735 -> beda8145c


GEODE-961 CI failure: PutAllCSDUnitTest.testPartialKeyInPRSingleHop

One of the clients was unnecessarily registering interest, allowing it to
get all of the entries created by the test if it happened to fail over from
one server to another during the test.


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

Branch: refs/heads/develop
Commit: beda8145c7c90c9d84bcb3b64ef2f16da13805c3
Parents: ffd9207
Author: Bruce Schuchardt <bschucha...@pivotal.io>
Authored: Wed Mar 30 09:21:54 2016 -0700
Committer: Bruce Schuchardt <bschucha...@pivotal.io>
Committed: Wed Mar 30 09:25:29 2016 -0700

----------------------------------------------------------------------
 .../gemfire/internal/cache/PutAllCSDUnitTest.java         | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/beda8145/geode-cq/src/test/java/com/gemstone/gemfire/internal/cache/PutAllCSDUnitTest.java
----------------------------------------------------------------------
diff --git 
a/geode-cq/src/test/java/com/gemstone/gemfire/internal/cache/PutAllCSDUnitTest.java
 
b/geode-cq/src/test/java/com/gemstone/gemfire/internal/cache/PutAllCSDUnitTest.java
index 0de00f0..505826e 100644
--- 
a/geode-cq/src/test/java/com/gemstone/gemfire/internal/cache/PutAllCSDUnitTest.java
+++ 
b/geode-cq/src/test/java/com/gemstone/gemfire/internal/cache/PutAllCSDUnitTest.java
@@ -2249,6 +2249,15 @@ public void testOneServer() throws CacheException, 
InterruptedException {
     stopBridgeServers(getCache());
   }
   
+  
+  public void testRepeat() throws Exception {
+    long giveup = System.currentTimeMillis() + (30 * 60000);
+    do {
+      testPartialKeyInPRSingleHop();
+      tearDown(); setUp();
+    } while (System.currentTimeMillis() < giveup);
+  }
+
   /**
    * Tests partial key putAll to 2 PR servers, because putting data at server
    * side is different between PR and LR. PR does it in postPutAll.
@@ -2358,7 +2367,6 @@ public void testOneServer() throws CacheException, 
InterruptedException {
       public void run2() throws CacheException {
         Region region = getRootRegion().getSubregion(regionName);
         region.getAttributesMutator().addCacheListener(new MyListener(false));
-        region.registerInterest("ALL_KEYS");
 
         // create keys
         try {

Reply via email to