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

Raghu Angadi commented on HADOOP-5832:
--------------------------------------

Thanks for the changes. Looks better. 

nit: Could you move the block of code that checks for consistency (including 
the check to see if it file://) a method? It is repeated for dirs and edits (it 
around 20 lines).


> Process dfs.name.edits.dirs as URI 
> -----------------------------------
>
>                 Key: HADOOP-5832
>                 URL: https://issues.apache.org/jira/browse/HADOOP-5832
>             Project: Hadoop Core
>          Issue Type: Sub-task
>          Components: dfs
>    Affects Versions: 0.20.0
>            Reporter: Luca Telloli
>             Fix For: 0.21.0
>
>         Attachments: HADOOP-5832.patch, HADOOP-5832.patch, HADOOP-5832.patch
>
>
> Process the value of property dfs.name.edits.dirs as URI, to allow different 
> schemes than just file. As an advantage, Java supports the constructor 
> File(URI) so the transition is straightforward for files. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to