[ 
https://issues.apache.org/jira/browse/GEODE-8645?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17219970#comment-17219970
 ] 

ASF GitHub Bot commented on GEODE-8645:
---------------------------------------

kohlmu-pivotal commented on a change in pull request #5655:
URL: https://github.com/apache/geode/pull/5655#discussion_r511197468



##########
File path: 
geode-redis/src/integrationTest/java/org/apache/geode/redis/internal/executor/pubsub/PubSubIntegrationTest.java
##########
@@ -16,13 +16,20 @@
 package org.apache.geode.redis.internal.executor.pubsub;
 
 import org.junit.ClassRule;
+import org.junit.contrib.java.lang.system.ProvideSystemProperty;
+import org.junit.rules.RuleChain;
 
 import org.apache.geode.redis.GeodeRedisServerRule;
 
 public class PubSubIntegrationTest extends AbstractPubSubIntegrationTest {
-  @ClassRule
+
   public static GeodeRedisServerRule server = new GeodeRedisServerRule();
 
+  @ClassRule
+  public static RuleChain chain = RuleChain.outerRule(
+      new ProvideSystemProperty("io.netty.eventLoopThreads", "10"))
+      .around(server);
+

Review comment:
       @kirklund would you be happy with the instead of the 
`RestoreSystemProperties` and the changing of the properties in a 
`@BeforeClass` block?




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Reduce netty threads used by PubSubIntegrationTest
> --------------------------------------------------
>
>                 Key: GEODE-8645
>                 URL: https://issues.apache.org/jira/browse/GEODE-8645
>             Project: Geode
>          Issue Type: Test
>          Components: redis, tests
>            Reporter: Jens Deppe
>            Priority: Major
>              Labels: pull-request-available
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to