[ https://issues.apache.org/jira/browse/GEODE-3969?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16257341#comment-16257341 ]
ASF GitHub Bot commented on GEODE-3969: --------------------------------------- galen-pivotal commented on a change in pull request #1047: GEODE-3969: Remove CallbackArgs from protobuf requests URL: https://github.com/apache/geode/pull/1047#discussion_r151751776 ########## File path: geode-protobuf/src/main/proto/v1/clientProtocol.proto ########## @@ -35,7 +35,6 @@ message Message { } message Request { - CallbackArguments callbackArg = 1; oneof requestAPI { PutRequest putRequest = 2; Review comment: I kind of want to keep the one available just in case, but I don't think it's actually a good idea. Probably we should decrement all the indices. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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 > Protobuf protocol requests includes a CallbackArgs field > -------------------------------------------------------- > > Key: GEODE-3969 > URL: https://issues.apache.org/jira/browse/GEODE-3969 > Project: Geode > Issue Type: Bug > Components: client/server > Reporter: Brian Rowe > > We eventually want to support allowing clients to pass arguments to cache > listeners in their requests, but this functionality is not going to be > present in v1. Having this field is therefore misleading and it should be > removed. -- This message was sent by Atlassian JIRA (v6.4.14#64029)