[GitHub] [geode] dschneider-pivotal commented on a change in pull request #5420: GEODE-8333: Second attempt to fix PUBSUB hang

dschneider-pivotal commented on a change in pull request #5420: URL: https://github.com/apache/geode/pull/5420#discussion_r468723671 ## File path: geode-redis/src/main/java/org/apache/geode/redis/internal/netty/ExecutionHandlerContext.java ## @@ -138,6 +161,25 @@ public void e

[GitHub] [geode] dschneider-pivotal commented on a change in pull request #5420: GEODE-8333: Second attempt to fix PUBSUB hang

dschneider-pivotal commented on a change in pull request #5420: URL: https://github.com/apache/geode/pull/5420#discussion_r468692943 ## File path: geode-redis/src/main/java/org/apache/geode/redis/internal/netty/ExecutionHandlerContext.java ## @@ -59,17 +60,21 @@ public class

[GitHub] [geode] dschneider-pivotal commented on a change in pull request #5420: GEODE-8333: Second attempt to fix PUBSUB hang

dschneider-pivotal commented on a change in pull request #5420: URL: https://github.com/apache/geode/pull/5420#discussion_r468688937 ## File path: geode-redis/src/main/java/org/apache/geode/redis/internal/netty/ExecutionHandlerContext.java ## @@ -138,6 +161,25 @@ public void e

[GitHub] [geode] dschneider-pivotal commented on a change in pull request #5420: GEODE-8333: Second attempt to fix PUBSUB hang

dschneider-pivotal commented on a change in pull request #5420: URL: https://github.com/apache/geode/pull/5420#discussion_r466037897 ## File path: geode-redis/src/distributedTest/java/org/apache/geode/redis/internal/executor/pubsub/PubSubDUnitTest.java ## @@ -86,11 +89,17 @@ p