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

Aaron T. Myers commented on HADOOP-7418:
----------------------------------------

Hey Andrew, patch looks pretty good to me. One question:

>From manually inspecting the code, I suspect the {{Path}} class will still not 
>correctly handle the case of multiple trailing slashes. Would you mind adding 
>a test case to test some path with a silly number (> 5) of trailing slashes?

You might also consider running the full HDFS and MR test suites with this 
patch installed, as those obviously exercise all of the {{Path}} code quite a 
bit. See the "Changes that span projects" section of this page for instructions 
about how to get HDFS and MR to compile against your patched Common: 
http://wiki.apache.org/hadoop/HowToContribute

> support for multiple slashes in the path separator
> --------------------------------------------------
>
>                 Key: HADOOP-7418
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7418
>             Project: Hadoop Common
>          Issue Type: Bug
>    Affects Versions: 0.23.0
>         Environment: Linux running JDK 1.6
>            Reporter: Sudharsan Sampath
>            Assignee: Andrew Look
>            Priority: Minor
>              Labels: newbie
>             Fix For: 0.23.0
>
>         Attachments: HDFS-1460.txt, HDFS-1460.txt
>
>
> the parsing of the input path string to identify the uri authority conflicts 
> with the file system paths. For instance the following is a valid path in 
> both the linux file system and the hdfs.
> //user/directory1//directory2.
> While this works perfectly fine in the command line for manipulating hdfs, 
> the same fails when specified as the input path for a mapper class with the 
> following expcetion.
> Exception in thread "main" java.net.UnknownHostException: unknown host: user
>         at org.apache.hadoop.ipc.Client$Connection.<init>(Client.java:195)
> as the org.apache.hadoop.fs.Path class assumes the string that follows the 
> '//' to be an uri authority

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to