simbadzina commented on code in PR #4311:
URL: https://github.com/apache/hadoop/pull/4311#discussion_r923567042


##########
hadoop-common-project/hadoop-common/src/main/proto/RpcHeader.proto:
##########
@@ -91,6 +91,7 @@ message RpcRequestHeaderProto { // the header for the 
RpcRequest
   optional RPCTraceInfoProto traceInfo = 6; // tracing info
   optional RPCCallerContextProto callerContext = 7; // call context
   optional int64 stateId = 8; // The last seen Global State ID
+  map<string, int64> nameserviceStateIds = 9; // Last seen state IDs for 
multiple nameservices.

Review Comment:
   On my laptop this compiles okay with protoc-2.5.0.
   https://developers.google.com/protocol-buffers/docs/proto#maps
   
   The protobuf v2 spec also show the map syntax.



-- 
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: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to