FrozenGene commented on a change in pull request #4281: [RUTNIME] Support C++ RPC URL: https://github.com/apache/incubator-tvm/pull/4281#discussion_r344478615
########## File path: src/runtime/rpc/rpc_session.h ########## @@ -36,8 +36,29 @@ namespace tvm { namespace runtime { +// Magic header for RPC data plane const int kRPCMagic = 0xff271; +// magic header for RPC tracker(control plane) +const int kRPCTrackerMagic = 0x2f271; +// sucess response +const int kRPCSuccess = kRPCMagic + 0; +// duplicate key in proxy +const int kRPCDupicate = kRPCMagic + 1; Review comment: It is used for proxy mode. However, this PR doesn't implement proxy in fact, so I remove related unnecessary code, currently work like our java standalone server. ---------------------------------------------------------------- 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 With regards, Apache Git Services