[ 
https://issues.apache.org/jira/browse/HADOOP-7776?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13140285#comment-13140285
 ] 

Doug Cutting commented on HADOOP-7776:
--------------------------------------

This looks good.

If you wanted to make this more easily extensible (HADOOP-7557) then you might 
change it to have a <<key,value> format, perhaps where keys are vints and 
values are <vint><payload>.  If you put the current three values (kind, op, id) 
then its size would grow from 6 to perhaps 10 bytes.  Alternately you could add 
a metadata field <count, <key,value>> for future use which would only add a 
single byte but wouldn't let you ever remove one of the initial three fields.

Per-call metadata would make it easier to add, e.g., Dapper-style tracing.
                
> Make the Ipc-Header in a RPC-Payload an explicit header
> -------------------------------------------------------
>
>                 Key: HADOOP-7776
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7776
>             Project: Hadoop Common
>          Issue Type: Sub-task
>            Reporter: Sanjay Radia
>            Assignee: Sanjay Radia
>         Attachments: RpcPayLoadHeader.java, rpcPayload1.patch, 
> rpcPayload2.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to