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

Tsz Wo (Nicholas), SZE commented on HDFS-2246:
----------------------------------------------

Comments on the client side:

- In BlockReaderLocal,
-* In getBlockPathInfo(..), use the local variable proxy in the following.  
Otherwise, there is a race condition that datanodeProxy could be null.
{code}
      pathinfo = datanodeProxy.getBlockLocalPathInfo(blk);
{code}
-* rename datanodeProxy to localhostProxy
-* replace the DatanodeInfo parameter newBlockReader with a port and use 
localhost:port to create proxy.
-* when should the proxy be stopped?
-* Why recordAllowedUser but not read the user from conf?


                
> Shortcut a local client reads to a Datanodes files directly
> -----------------------------------------------------------
>
>                 Key: HDFS-2246
>                 URL: https://issues.apache.org/jira/browse/HDFS-2246
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>            Reporter: Sanjay Radia
>         Attachments: 0001-HDFS-347.-Local-reads.patch, 
> HDFS-2246-branch-0.20-security-205.patch, 
> HDFS-2246-branch-0.20-security-205.patch, 
> HDFS-2246-branch-0.20-security.patch, HDFS-2246.20s.1.patch, 
> HDFS-2246.20s.2.txt, HDFS-2246.20s.3.txt, HDFS-2246.20s.4.txt, 
> HDFS-2246.20s.patch, localReadShortcut20-security.2patch
>
>


--
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