[
https://issues.apache.org/jira/browse/HADOOP-7557?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13126040#comment-13126040
]
Sanjay Radia commented on HADOOP-7557:
--------------------------------------
The only extra field added was the protocolKind. Our RPC headers are not well
designed.
ProtocolKind is necessary since the next layer currently supports two different
serializations and can support additional ones.
As far as smaller patch -- that is mostly due to auto generated files from PB.
HDFS, since Todd's data transfer PB stuff, has chosen to commit the PB
generated files rather than generate them via mvn.
If I were to add a new field to the main header (the "hrpc" one) to indicate
the format of the next layer would it satisfy your concerns? It would allow one
to write all the layers in Writables, or PB or Avro.
> Make IPC header be extensible
> -------------------------------
>
> Key: HADOOP-7557
> URL: https://issues.apache.org/jira/browse/HADOOP-7557
> Project: Hadoop Common
> Issue Type: Improvement
> Reporter: Sanjay Radia
> Assignee: Sanjay Radia
> Attachments: HADOOP-7557.patch, IpcHeader.proto, ipcHeader1.patch,
> ipcHeader2.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
