alex-plekhanov commented on a change in pull request #9629:
URL: https://github.com/apache/ignite/pull/9629#discussion_r776575006



##########
File path: 
modules/core/src/main/java/org/apache/ignite/internal/processors/platform/client/ClientMessageParser.java
##########
@@ -277,7 +277,6 @@
     private static final short OP_SERVICE_GET_DESCRIPTOR = 7002;
 
     /** Data streamers. */
-    /** */

Review comment:
       Return this

##########
File path: 
modules/core/src/main/java/org/apache/ignite/internal/processors/platform/client/ClientMessageParser.java
##########
@@ -491,7 +490,8 @@ public ClientListenerRequest decode(BinaryReaderExImpl 
reader) {
                 return new ClientExecuteTaskRequest(reader);
 
             case OP_SERVICE_INVOKE:
-                return new ClientServiceInvokeRequest(reader);
+                return new ClientServiceInvokeRequest(reader,

Review comment:
       Let's pass `protocolCtx` to `ClientServiceInvokeRequest` (see other 
requests as example `ClientCacheGetConfigurationRequest`, 
`ClientCacheSqlFieldsQueryRequest`)




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