[geode-benchmarks] branch gcpscaletest updated: going back to clearing valuemap

[wi...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

wirebaron pushed a commit to branch gcpscaletest
in repository https://gitbox.apache.org/repos/asf/geode-benchmarks.git


The following commit(s) were added to refs/heads/gcpscaletest by this push:
     new 166923d  going back to clearing valuemap
166923d is described below

commit 166923df18b348be3da9b13597977f430f63759a
Author: Brian Rowe <br...@pivotal.io>
AuthorDate: Fri Dec 21 11:42:54 2018 -0800

    going back to clearing valuemap
---
 .../main/java/org/apache/geode/benchmark/tasks/PrePopulateRegion.java  | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tasks/PrePopulateRegion.java b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tasks/PrePopulateRegion.java
index cce2973..bae2237 100644
--- a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tasks/PrePopulateRegion.java
+++ b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tasks/PrePopulateRegion.java
@@ -126,11 +126,12 @@ public class PrePopulateRegion implements Task {
     Map<Long, Portfolio> valueMap = new HashMap<>();
     for (long putIndex = lowBound; putIndex < highBound; putIndex++) {
       // build a map of to put to the server
-      valueMap = new HashMap<>();
+
       valueMap.put(putIndex, new Portfolio(putIndex));
 
       if (putIndex % getBatchSize() == 0) {
         region.putAll(valueMap);
+        valueMap.clear();
       }
     }