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