You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by qi...@apache.org on 2015/08/06 19:11:19 UTC

[08/19] incubator-geode git commit: GEODE-95:PartitionedRegionSingleHopDUnitTest.test_MetadataContents fails intermittently There is a race between updating the metadata by client and creation of secondary bucket. Once, the client fetches the metadata w

GEODE-95:PartitionedRegionSingleHopDUnitTest.test_MetadataContents fails intermittently
 There is a race between updating the metadata by client and creation of secondary bucket.
Once, the client fetches the metadata without information of secondary bucket(due to race), any further put/get will go to the correct server which hosts the bucket for which client has information. This will not cause any re-fetching of metadata from server as there is no hop. Hence only making test changes.


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

Branch: refs/heads/feature/GEODE-137
Commit: 787b69ded2269dd3dea388df222db8fd9f9d2789
Parents: 0a555b9
Author: kbachhav <kbachhav.pivotal.io>
Authored: Thu Jul 30 18:46:08 2015 +0530
Committer: Qihong Chen <qc...@pivotal.io>
Committed: Thu Aug 6 10:07:47 2015 -0700

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


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/787b69de/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 95fbe70..39618bb 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
@@ -2216,10 +2216,6 @@ public class PartitionedRegionSingleHopDUnitTest extends CacheTestCase {
       }
     };
     DistributedTestCase.waitForCriterion(wc, 60000, 1000, true);
-    for (Entry entry : prMetaData.getBucketServerLocationsMap_TEST_ONLY()
-        .entrySet()) {
-      assertEquals(2, ((List)entry.getValue()).size());
-    }
   }
 }