Bill commented on a change in pull request #7449:
URL: https://github.com/apache/geode/pull/7449#discussion_r835648823
##########
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;
- // number of concurrent (sending) tasks to run
- private static final int SENDER_COUNT = 10;
+ // number of concurrent (sending) tasks to run in each sending JVM
+ private static final int TASKS_PER_SENDER = 10;
- // (exclusive) upper bound of random message size, in bytes
- private static final int LARGEST_MESSAGE_BOUND = 32 * 1024 + 2; // 32KiB + 2
+ // how many messages will each sending task generate?
+ private static final int MESSAGES_PER_SENDING_TASK = 1_000;
+
+ /*
+ * Upper bound (exclusive) of random message size, in bytes.
+ * The magnitude is chosen to give us a mix of messages around
+ * 16KB. I want a number that results in a largest message that
+ * is not a power of two since I believe, for no particular reason,
+ * that it might catch more bugs than a power of two would,
+ * so I add 2.
+ */
+ private static final int LARGEST_MESSAGE_BOUND = 32 * 1024 + 2;
+
+ /*
+ * Non-random payload content makes debugging easier when a NULL cipher is
used.
+ * Null ciphers are not allowed on TLSv1.3 though.
+ */
+ private static boolean RANDOMIZE_PAYLOAD_CONTENT = false;
Review comment:
In working on this ticket I tried to use a NULL cipher. When using a
NULL cipher it's really useful to have a non-random pattern for the payload so
you can see, e.g. in packet captures, the payload bytes. In its final form, we
are not using a NULL cipher, but the ability to switch between random and
non-random content is useful 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-unsubscr...@geode.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
