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



##########
File path: 
geode-wan/src/distributedTest/java/org/apache/geode/management/internal/cli/commands/AlterRegionCommandWithRemoteLocator.java
##########
@@ -207,8 +207,34 @@ public void 
whenAlteringMultipleRegionWithAlterCommandToAddGatewaySendersThenItS
           "alter region --name=SecurityCrossReferences 
--gateway-sender-id=serialSender1,serialSender2");
       return true;
     });
+  }
 
+  /**
+   * Test execution of alter region command, when adding gw sender to already 
initialized region is
+   * not taking too much time.
+   * Verify that execution of command will not be halted, due to 
PRShadowRegion not available
+   * immediately in all servers.
+   */
+  @Test
+  public void 
alterInitializedRegionWithGwSenderOnManyServersDoesNotTakeTooLong() {

Review comment:
       Also, this is test that reproduce fault in old PR. You can repeat test 
case several times, it will fail.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to