geode-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From upthewatersp...@apache.org
Subject incubator-geode git commit: GEODE-139 Fixing RebalanceOperationDUnitTest.testWaitFor* methods
Date Wed, 26 Aug 2015 15:51:07 GMT
Repository: incubator-geode
Updated Branches:
  refs/heads/develop 01e8b5ecd -> c3695aa91


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/develop
Commit: c3695aa912e4ff5b32f07e0eb312b9cedf749ebb
Parents: 01e8b5e
Author: Dan Smith <upthewaterspout@apache.org>
Authored: Wed Aug 26 08:47:56 2015 -0700
Committer: Dan Smith <upthewaterspout@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


Mime
View raw message