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

Tsz Wo (Nicholas), SZE commented on HADOOP-9716:
------------------------------------------------


> ... Currently in Rpc response I did not add the client Id. ...

Yes, we do not have client id but we already have callId in 
RpcResponseHeaderProto.  Client id is not needed since the client receiving the 
response knows its own id.

> Do you mean add it to the response? ...

I suggest also adding retry number to both RpcRequestHeaderProto and 
RpcResponseHeaderProto since, currently, call id uniquely identifies calls 
including retires.  If we use the same call id for retries, then we need call 
id + retry number to uniquely identifies them.  Otherwise, there will be more 
change as you suggested earlier.
                
> Move the Rpc request call ID generation to client side InvocationHandler
> ------------------------------------------------------------------------
>
>                 Key: HADOOP-9716
>                 URL: https://issues.apache.org/jira/browse/HADOOP-9716
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: ipc
>            Reporter: Suresh Srinivas
>            Assignee: Tsz Wo (Nicholas), SZE
>         Attachments: c9716_20130712b.patch, c9716_20130712.patch, 
> c9716_20130713b.patch, c9716_20130713c.patch, c9716_20130713.patch
>
>
> Currently when RetryInvocationHandler is used to retry an RPC request, a new 
> RPC request call ID is generated. This jira proposes moving call ID 
> generation to InvocationHandler so that retried RPC requests retain the same 
> call ID. This is needed for RetryCache functionality proposed in HDFS-4942.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to