You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by es...@apache.org on 2015/10/20 01:26:30 UTC

incubator-geode git commit: Fixes GEODE-444 by closing client cache before closing server cache.

Repository: incubator-geode
Updated Branches:
  refs/heads/feature/GEODE-409 5da17d120 -> 26fbf659c


Fixes GEODE-444 by closing client cache before closing server cache.


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

Branch: refs/heads/feature/GEODE-409
Commit: 26fbf659c21e5a3584bbbdaa33060af68266a2bc
Parents: 5da17d1
Author: eshu <es...@pivotal.io>
Authored: Mon Oct 19 16:21:07 2015 -0700
Committer: eshu <es...@pivotal.io>
Committed: Mon Oct 19 16:26:18 2015 -0700

----------------------------------------------------------------------
 .../internal/cache/PartitionedRegionSingleHopDUnitTest.java  | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/26fbf659/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/PartitionedRegionSingleHopDUnitTest.java
----------------------------------------------------------------------
diff --git a/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/PartitionedRegionSingleHopDUnitTest.java b/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/PartitionedRegionSingleHopDUnitTest.java
index 39618bb..278c477 100755
--- a/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/PartitionedRegionSingleHopDUnitTest.java
+++ b/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/PartitionedRegionSingleHopDUnitTest.java
@@ -106,14 +106,14 @@ public class PartitionedRegionSingleHopDUnitTest extends CacheTestCase {
 
   public void tearDown2() throws Exception {
     try {
-
+      /* fixes GEODE-444, really close client cache first by using super.tearDown2();
       // close the clients first
       member0.invoke(PartitionedRegionSingleHopDUnitTest.class, "closeCache");
       member1.invoke(PartitionedRegionSingleHopDUnitTest.class, "closeCache");
       member2.invoke(PartitionedRegionSingleHopDUnitTest.class, "closeCache");
       member3.invoke(PartitionedRegionSingleHopDUnitTest.class, "closeCache");
       closeCache();
-
+      */
       super.tearDown2();
 
       member0 = null;
@@ -755,7 +755,7 @@ public class PartitionedRegionSingleHopDUnitTest extends CacheTestCase {
     pause(5000);
     assertFalse(cms.isRefreshMetadataTestOnly());
   }
-  
+
   public void testServerLocationRemovalThroughPing() {
     Integer port0 = (Integer)member0.invoke(
         PartitionedRegionSingleHopDUnitTest.class, "createServer",
@@ -842,7 +842,7 @@ public class PartitionedRegionSingleHopDUnitTest extends CacheTestCase {
     DistributedTestCase.waitForCriterion(wc, 60000, 1000, true);
 //    assertEquals(4/*numBuckets*/, prMetaData.getBucketServerLocationsMap_TEST_ONLY().size());    
   }
-  
+
   public void testMetadataFetchOnlyThroughputAll() {
     Integer port0 = (Integer)member0.invoke(
         PartitionedRegionSingleHopDUnitTest.class, "createServer",