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



##########
File path: 
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/command/GatewayReceiverCommand.java
##########
@@ -350,6 +352,7 @@ public void cmdExecute(final Message clientMessage, final 
ServerConnection serve
               break;
 
             case 1: // Update
+            case GatewaySenderEventImpl.UPDATE_ACTION_NO_GENERATE_CALLBACKS:
               try {

Review comment:
       Alternatively, we could make the command fail if the remote side is in a 
version that does not support the command or  send a regular UPDATE action 
assuming that callbacks will be invoked. I think the first alternative makes 
more sense.




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