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 2015/09/01 18:49:00 UTC

[22/37] incubator-geode git commit: GEODE-139 Fixing RebalanceOperationDUnitTest.testWaitFor* methods

GEODE-139 Fixing RebalanceOperationDUnitTest.testWaitFor* methods

The issue is that a previous test method was polluting the system. There
is a test of redundancy zones that sets the redundancy zone in 4 members
and disconnects the distributed system in all of those members at the
end.

However, there is a 5th member, the controller VM, that still has a
distributed system. It never had a redundancy zone set but it still
flipped a bit to indicate that redundancy zones should be enforced. From
that point on we refused to satisfy redundancy.


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

Branch: refs/heads/feature/GEODE-77
Commit: c3695aa912e4ff5b32f07e0eb312b9cedf749ebb
Parents: 01e8b5e
Author: Dan Smith <up...@apache.org>
Authored: Wed Aug 26 08:47:56 2015 -0700
Committer: Dan Smith <up...@apache.org>
Committed: Wed Aug 26 08:47:56 2015 -0700

----------------------------------------------------------------------
 .../gemfire/internal/cache/control/RebalanceOperationDUnitTest.java | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/c3695aa9/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/control/RebalanceOperationDUnitTest.java
----------------------------------------------------------------------
diff --git a/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/control/RebalanceOperationDUnitTest.java b/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/control/RebalanceOperationDUnitTest.java
index 819bf9e..1b04ecb 100644
--- a/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/control/RebalanceOperationDUnitTest.java
+++ b/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/control/RebalanceOperationDUnitTest.java
@@ -753,6 +753,7 @@ public class RebalanceOperationDUnitTest extends CacheTestCase {
     checkBucketCount(vm1, "region2", 3);
     checkBucketCount(vm2, "region2", 6);
     } finally {
+      disconnectFromDS();
       invokeInEveryVM(new SerializableRunnable() {
         public void run() {
           //clear the redundancy zone setting