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

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


The following commit(s) were added to refs/heads/develop by this push:
     new d54083c7d7 GEODE-10197: fix test for jdk17
d54083c7d7 is described below

commit d54083c7d7b0d14631df262964ef071599660666
Author: Darrel Schneider <dar...@vmware.com>
AuthorDate: Fri Apr 15 23:41:32 2022 -0700

    GEODE-10197: fix test for jdk17
    
    OutOfMemoryDUnitTest now only uses CMS on jdk less than 14. (#7595)
---
 .../java/org/apache/geode/redis/OutOfMemoryDUnitTest.java           | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git 
a/geode-for-redis/src/distributedTest/java/org/apache/geode/redis/OutOfMemoryDUnitTest.java
 
b/geode-for-redis/src/distributedTest/java/org/apache/geode/redis/OutOfMemoryDUnitTest.java
index 5c49948dbd..a1a81e89ea 100644
--- 
a/geode-for-redis/src/distributedTest/java/org/apache/geode/redis/OutOfMemoryDUnitTest.java
+++ 
b/geode-for-redis/src/distributedTest/java/org/apache/geode/redis/OutOfMemoryDUnitTest.java
@@ -15,6 +15,8 @@
 
 package org.apache.geode.redis;
 
+import static org.apache.commons.lang3.JavaVersion.JAVA_13;
+import static org.apache.commons.lang3.SystemUtils.isJavaVersionAtMost;
 import static 
org.apache.geode.internal.AvailablePortHelper.getRandomAvailableTCPPorts;
 import static 
org.apache.geode.management.internal.i18n.CliStrings.START_LOCATOR;
 import static 
org.apache.geode.management.internal.i18n.CliStrings.START_LOCATOR__DIR;
@@ -143,8 +145,10 @@ public class OutOfMemoryDUnitTest {
         .addOption(START_SERVER__INITIAL_HEAP, "125m")
         .addOption(START_SERVER__MAXHEAP, "125m")
         .addOption(START_SERVER__CRITICAL__HEAP__PERCENTAGE, "50")
-        .addOption(START_SERVER__J, "-XX:CMSInitiatingOccupancyFraction=45")
         .addOption(START_SERVER__CLASSPATH, redisHome.getGeodeForRedisHome() + 
"/lib/*");
+    if (isJavaVersionAtMost(JAVA_13)) {
+      startServerCommand.addOption(START_SERVER__J, 
"-XX:CMSInitiatingOccupancyFraction=45");
+    }
     
gfsh.executeAndAssertThat(startServerCommand.getCommandString()).statusIsSuccess();
   }
 

Reply via email to