mivanac commented on a change in pull request #7347:
URL: https://github.com/apache/geode/pull/7347#discussion_r837732797



##########
File path: 
geode-assembly/src/acceptanceTest/java/org/apache/geode/cache/wan/SeveralGatewayReceiversWithSamePortAndHostnameForSendersTest.java
##########
@@ -215,6 +216,55 @@ public void 
testTwoSendersWithSameIdShouldUseSameValueForEnforceThreadsConnectTo
 
   }
 
+
+
+  /**
+   * The aim of this test is verify that when several gateway receivers in a 
remote site share the
+   * same port and hostname-for-senders, the pings sent from the gateway 
senders reach the right
+   * gateway receiver and not just any of the receivers. Failure to do this 
may result in the
+   * closing of connections by a gateway receiver for not having received the 
ping in time.
+   */
+  @Test
+  public void 
testPingsToReceiversWithSamePortAndHostnameForSendersReachTheRightReceiver()

Review comment:
       Thanks for comments. This test is different. It uses 1 dispatcher 
thread, and it needs to prove that only one endpoint will be pinged. Previously 
this was not the case. I will update comment.




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


Reply via email to