You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ds...@apache.org on 2015/07/25 01:15:34 UTC
[11/50] 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
[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-86
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) {