[geode-benchmarks] branch sni updated: turn down threads to 1 to troubleshoot client-server comms

[burcham]

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

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


The following commit(s) were added to refs/heads/sni by this push:
     new 7611069  turn down threads to 1 to troubleshoot client-server comms
7611069 is described below

commit 76110693ffe667193d6cc157595a1f530d66d0b1
Author: Bill Burcham <bburc...@pivotal.io>
AuthorDate: Thu May 28 10:53:31 2020 -0700

    turn down threads to 1 to troubleshoot client-server comms
---
 README.md                                                              | 3 ++-
 .../main/java/org/apache/geode/benchmark/tasks/PrePopulateRegion.java  | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/README.md b/README.md
index 237f777..9918834 100644
--- a/README.md
+++ b/README.md
@@ -164,7 +164,8 @@ Also we have to provide `-DwithSsl=true` for an SNI test 
even though no SNI test
 * ~~generate `haproxy.cfg` with client-visible SNI hostnames~~
 * ~~turn on SNI via `setPoolSocketFactory` in a new `StartClientSNI` task~~
 * ~~set `--hostname-for-clients` on locator and servers for SNI~~
-* remove `-Djavax.net.debug=all` once connections are working
+* set `numThreads` back to number of cores in `PrePopulateRegion` task after 
client-server connections are working
+* remove `-Djavax.net.debug=all` once client-server connections are working
 * turn log-level back to "config" for server and locator 
 * make topology orthogonal to tests so all tests can run with SNI; have a 
`-Psni`/`-Dsni` flag
 * fix borken `PartitionedPutBenchmarkSNITest`: 
`DefineHostNamingsOffPlatformTask` breaks when running multiple roles on a 
single host
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 faae4d7..4ed2a89 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
@@ -77,7 +77,7 @@ public class PrePopulateRegion implements Task {
     logger.info("*******************************************");
     final Instant start = Instant.now();
 
-    final int numThreads = Runtime.getRuntime().availableProcessors();
+    final int numThreads = 1; //Runtime.getRuntime().availableProcessors();
     final ExecutorService threadPool = 
Executors.newFixedThreadPool(numThreads);
     final List<CompletableFuture<Void>> futures = new ArrayList<>();