You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by bs...@apache.org on 2016/03/30 18:25:53 UTC
incubator-geode git commit: GEODE-961 CI failure:
PutAllCSDUnitTest.testPartialKeyInPRSingleHop
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 <bs...@pivotal.io>
Authored: Wed Mar 30 09:21:54 2016 -0700
Committer: Bruce Schuchardt <bs...@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 {