You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2022/03/25 22:24:49 UTC

[GitHub] [geode] Bill commented on a change in pull request #7449: GEODE-10122: P2P Handles TLS KeyUpdate Message

Bill commented on a change in pull request #7449:
URL: https://github.com/apache/geode/pull/7449#discussion_r835648095



##########
File path: geode-core/src/distributedTest/java/org/apache/geode/distributed/internal/P2PMessagingConcurrencyDUnitTest.java
##########
@@ -70,28 +77,71 @@
 @RunWith(GeodeParamsRunner.class)
 public class P2PMessagingConcurrencyDUnitTest {
 
-  // how many messages will each sender generate?
-  private static final int MESSAGES_PER_SENDER = 1_000;
+  // how many sending member JVMs
+  private static final int SENDERS = 1;

Review comment:
       In the course of working on this ticket, I enhanced this test so that the number of sending member/JVMs could be dialed up (larger than one). I am leaving it set at 1 for CI but wanted to leave in the ability to dial it up when we need it, for troubleshooting.




-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

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