GEODE-8577: Fix flaky PubSubNativeRedisAcceptanceTest (apache#5597)

- Apparently we need more time between retrying for a new connection on docker.
IronOnet · Oct 6, 2020 · f8dae61 · f8dae61
1 parent 9bc288a
commit f8dae61
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/...java/org/apache/geode/redis/internal/executor/pubsub/PubSubNativeRedisAcceptanceTest.java b/...java/org/apache/geode/redis/internal/executor/pubsub/PubSubNativeRedisAcceptanceTest.java
@@ -16,11 +16,9 @@
 package org.apache.geode.redis.internal.executor.pubsub;
 
 import org.junit.ClassRule;
-import org.junit.Ignore;
 
 import org.apache.geode.NativeRedisTestRule;
 
-@Ignore("GEODE-8577")
 public class PubSubNativeRedisAcceptanceTest extends AbstractPubSubIntegrationTest {
   @ClassRule
   public static NativeRedisTestRule redis = new NativeRedisTestRule();

diff --git a/...t/java/org/apache/geode/redis/internal/executor/pubsub/AbstractPubSubIntegrationTest.java b/...t/java/org/apache/geode/redis/internal/executor/pubsub/AbstractPubSubIntegrationTest.java
@@ -759,7 +759,7 @@ private Jedis getConnection() {
         }
       }
       try {
-        Thread.sleep(100);
+        Thread.sleep(1000);
       } catch (InterruptedException e) {
         e.printStackTrace();
       }