[geode-benchmarks] branch develop updated: GEODE-10126: Use system properties to configure redis

[upthewaterspout]

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

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


The following commit(s) were added to refs/heads/develop by this push:
     new 0892daf6 GEODE-10126: Use system properties to configure redis
0892daf6 is described below

commit 0892daf69939fc5c508856f787690dd444461c40
Author: Dan Smith <dasm...@vmware.com>
AuthorDate: Wed Apr 6 09:05:04 2022 -0700

    GEODE-10126: Use system properties to configure redis
    
    Use system properties to configure the redis server, instead of the gemfire
    properties, which have now been removed.
---
 .../geode/benchmark/redis/tasks/StartGeodeServer.java   | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git 
a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/redis/tasks/StartGeodeServer.java
 
b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/redis/tasks/StartGeodeServer.java
index 4b86dafe..f379d028 100644
--- 
a/geode-benchmarks/src/main/java/org/apache/geode/benchmark/redis/tasks/StartGeodeServer.java
+++ 
b/geode-benchmarks/src/main/java/org/apache/geode/benchmark/redis/tasks/StartGeodeServer.java
@@ -21,7 +21,6 @@ package org.apache.geode.benchmark.redis.tasks;
 import static java.lang.String.valueOf;
 
 import org.apache.geode.benchmark.tasks.StartServer;
-import org.apache.geode.cache.CacheFactory;
 import org.apache.geode.cache.server.CacheServer;
 import org.apache.geode.perftest.TestContext;
 
@@ -35,13 +34,15 @@ public class StartGeodeServer extends StartServer {
   }
 
   @Override
-  protected CacheFactory configureCacheFactory(final CacheFactory cacheFactory,
-      final TestContext context)
-      throws Exception {
-
-    return super.configureCacheFactory(cacheFactory, context)
-        .set("geode-for-redis-enabled", valueOf(true))
-        .set("geode-for-redis-port", valueOf(redisPort));
+  public void run(TestContext context) throws Exception {
+    try {
+      System.setProperty("gemfire.geode-for-redis-port", valueOf(redisPort));
+      System.setProperty("gemfire.geode-for-redis-enabled", valueOf(true));
+      super.run(context);
+    } finally {
+      System.clearProperty("gemfire.geode-for-redis-port");
+      System.clearProperty("gemfire.geode-for-redis-enabled");
+    }
   }
 
   @Override