[geode-benchmarks] branch GEODE-7554 updated: exit in a different place

[he...@apache.org]

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

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


The following commit(s) were added to refs/heads/GEODE-7554 by this push:
     new e4534fd  exit in a different place
e4534fd is described below

commit e4534fd17823f241380155d2a0b0be5367bb0518
Author: Robert Houghton <rh...@pivotal.io>
AuthorDate: Fri Dec 6 11:03:19 2019 -0800

    exit in a different place
    
    Signed-off-by: Helena A. Bales <hb...@pivotal.io>
---
 .../src/main/java/org/apache/geode/benchmark/tasks/NoopTask.java     | 5 ++++-
 .../main/java/org/apache/geode/benchmark/tests/NoopBenchmark.java    | 2 --
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tasks/NoopTask.java b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tasks/NoopTask.java
index b6da239..2ce000e 100644
--- a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tasks/NoopTask.java
+++ b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tasks/NoopTask.java
@@ -17,6 +17,8 @@
 
 package org.apache.geode.benchmark.tasks;
 
+import static java.lang.System.exit;
+
 import java.io.Serializable;
 import java.util.Map;
 
@@ -27,6 +29,7 @@ public class NoopTask extends BenchmarkDriverAdapter implements Serializable {
 
   @Override
   public boolean test(Map<Object, Object> ctx) {
-    return true;
+    exit(1);
+    return false;
   }
 }
diff --git a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/NoopBenchmark.java b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/NoopBenchmark.java
index 8609bf1..8d4e612 100644
--- a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/NoopBenchmark.java
+++ b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/tests/NoopBenchmark.java
@@ -18,7 +18,6 @@
 package org.apache.geode.benchmark.tests;
 
 
-import static java.lang.System.exit;
 import static org.apache.geode.benchmark.topology.ClientServerTopology.Roles.CLIENT;
 
 import org.junit.jupiter.api.Test;
@@ -43,7 +42,6 @@ public class NoopBenchmark implements PerformanceTest {
 
   @Override
   public TestConfig configure() {
-    exit(1);
     TestConfig config = GeodeBenchmark.createConfig();
     ClientServerTopology.configure(config);
     config.workload(new NoopTask(), CLIENT);