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



##########
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:
       For sure, this would fail with changed timeout, but that cannot be 
changed. For details you can see comments in previous PR.




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