geode-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jasonhu...@apache.org
Subject [16/37] incubator-geode git commit: [fixes #50618] : SingleHopGetAllPutAllDUnitTest: testServerGetAllFunction fails in CC r47313 -Key's are formed based on string "execKey" + integer. These keys does not guarantee that they wil be mapped to 13 configur
Date Thu, 23 Jul 2015 16:18:53 GMT
[fixes #50618] : SingleHopGetAllPutAllDUnitTest: testServerGetAllFunction fails in CC r47313
  -Key's are formed based on string "execKey" + integer. These keys does not guarantee that
they wil be mapped to 13 configured buckets.
  -Now instead of String keys, integer will be uased as key whch will guarantee that it will
map to configured number of buckets.

Ran:
 SingleHopGetAllPutAllDUnitTest multiple times


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

Branch: refs/heads/feature/GEODE-9
Commit: ad038b3b7363fd80a559292296f74a1d3bd13ace
Parents: 065072e
Author: kbachhav <kbachhav.pivotal.io>
Authored: Tue Jul 14 18:47:43 2015 +0530
Committer: kbachhav <kbachhav.pivotal.io>
Committed: Tue Jul 14 18:47:43 2015 +0530

----------------------------------------------------------------------
 .../execute/SingleHopGetAllPutAllDUnitTest.java   | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/ad038b3b/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/SingleHopGetAllPutAllDUnitTest.java
----------------------------------------------------------------------
diff --git a/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/SingleHopGetAllPutAllDUnitTest.java
b/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/SingleHopGetAllPutAllDUnitTest.java
index 9914032..fdbf148 100644
--- a/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/SingleHopGetAllPutAllDUnitTest.java
+++ b/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/execute/SingleHopGetAllPutAllDUnitTest.java
@@ -46,7 +46,7 @@ public class SingleHopGetAllPutAllDUnitTest extends PRClientServerTestBase{
    * hosting the data. 
    */
   @Ignore("Disabled due to bug #50618")
-  public void DISABLED_testServerGetAllFunction(){
+  public void testServerGetAllFunction(){
     createScenario();
     client.invoke(SingleHopGetAllPutAllDUnitTest.class,
         "getAll");
@@ -60,17 +60,19 @@ public class SingleHopGetAllPutAllDUnitTest extends PRClientServerTestBase{
   public static void getAll() {
     Region region = cache.getRegion(PartitionedRegionName);
     assertNotNull(region);
-    final List testKeysList = new ArrayList();
+    final List testValueList = new ArrayList();
+    final List testKeyList = new ArrayList();
     for (int i = (totalNumBuckets.intValue() * 3); i > 0; i--) {
-      testKeysList.add("execKey-" + i);
+      testValueList.add("execKey-" + i);    
     }
     DistributedSystem.setThreadsSocketPolicy(false);
     try {
       int j = 0;
       Map origVals = new HashMap();
-      for (Iterator i = testKeysList.iterator(); i.hasNext();) {
-        Integer val = new Integer(j++);
-        Object key = i.next();
+      for (Iterator i = testValueList.iterator(); i.hasNext();) {
+        testKeyList.add(j);
+        Integer key = new Integer(j++);
+        Object val = i.next();
         origVals.put(key, val);
         region.put(key, val);
       }
@@ -79,10 +81,10 @@ public class SingleHopGetAllPutAllDUnitTest extends PRClientServerTestBase{
       verifyMetadata();
       
       // check if the function was routed to pruned nodes
-      Map resultMap = region.getAll(testKeysList);
+      Map resultMap = region.getAll(testKeyList);
       assertTrue(resultMap.equals(origVals));
       pause(2000);
-      Map secondResultMap = region.getAll(testKeysList);
+      Map secondResultMap = region.getAll(testKeyList);
       assertTrue(secondResultMap.equals(origVals));
     }
     catch (Exception e) {


Mime
View raw message