[geode-benchmarks] branch sni updated: really reduce threads to 1

[bu...@apache.org]

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 743fbff  really reduce threads to 1
743fbff is described below

commit 743fbff5349e8148a77e945fe3f95a721d15625b
Author: Bill Burcham <bb...@pivotal.io>
AuthorDate: Mon Jun 1 14:33:04 2020 -0700

    really reduce threads to 1
---
 README.md                                                               | 2 +-
 .../java/org/apache/geode/benchmark/tests/PartitionedPutBenchmark.java  | 2 ++
 .../org/apache/geode/benchmark/tests/PartitionedPutBenchmarkSNI.java    | 2 ++
 3 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 29887d4..6265c4f 100644
--- a/README.md
+++ b/README.md
@@ -164,7 +164,7 @@ 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~~
-* reinstate thread-per-core in `PrePopulateRegion.run()`
+* reinstate thread-per-core in `PrePopulateRegion.run()` and in `PartitionedPutBenchmark[SNI]`
 * set `keyRange` back to 1e6 in `PartitionedPutBenchmark[SNI]` after client-server connections are healthy
 * 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/tests/PartitionedPutBenchmark.java b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/PartitionedPutBenchmark.java
index 46bf98f..5e803ed 100644
--- a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/PartitionedPutBenchmark.java
+++ b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/PartitionedPutBenchmark.java
@@ -57,6 +57,8 @@ public class PartitionedPutBenchmark implements PerformanceTest {
   @Override
   public TestConfig configure() {
     TestConfig config = GeodeBenchmark.createConfig();
+    // TODO: delete this once test is working!
+    config.threads(1);
     ClientServerTopology.configure(config);
     before(config, new CreatePartitionedRegion(), SERVER);
     before(config, new CreateClientProxyRegion(), CLIENT);
diff --git a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/PartitionedPutBenchmarkSNI.java b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/PartitionedPutBenchmarkSNI.java
index 5a8fdf6..617c1d1 100644
--- a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/PartitionedPutBenchmarkSNI.java
+++ b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/PartitionedPutBenchmarkSNI.java
@@ -59,6 +59,8 @@ public class PartitionedPutBenchmarkSNI implements PerformanceTest {
   @Override
   public TestConfig configure() {
     TestConfig config = GeodeBenchmark.createConfig();
+    // TODO: delete this once test is working!
+    config.threads(1);
     ClientServerTopologyWithSNIProxy.configure(config);
     before(config, new CreatePartitionedRegion(), SERVER);
     before(config, new CreateClientProxyRegion(), CLIENT);