albertogpz commented on a change in pull request #5348:
URL: https://github.com/apache/geode/pull/5348#discussion_r465912609



##########
File path: 
geode-gfsh/src/main/java/org/apache/geode/management/internal/cli/commands/StartGatewaySenderCommand.java
##########
@@ -155,4 +160,38 @@ public ResultModel startGatewaySender(@CliOption(key = 
CliStrings.START_GATEWAYS
 
     return resultModel;
   }
+
+  private boolean sendAllStartingMessage(Set<DistributedMember> dsMembers, 
Cache cache,

Review comment:
       Do you have an example from other part of the code where it is done as 
you are suggesting?
   Do you mean executing the function once for each member or having a function 
that executes on all members (the latter would seem more convenient)?




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

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


Reply via email to