[jira] [Commented] (HDFS-8038) PBImageDelimitedTextWriter#getEntry output HDFS path in platform-specific format.

[Hudson (JIRA)]

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

Hudson commented on HDFS-8038:
------------------------------

SUCCESS: Integrated in Hadoop-trunk-Commit #7523 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/7523/])
HDFS-8038. PBImageDelimitedTextWriter#getEntry output HDFS path in 
platform-specific format. Contributed by Xiaoyu Yao. (cnauroth: rev 
1e72d98c69bef3526cf0eb617de69e0b6d2fc13c)
* hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
* 
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/PBImageTextWriter.java
* 
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/PBImageDelimitedTextWriter.java


> PBImageDelimitedTextWriter#getEntry output HDFS path in platform-specific 
> format.
> ---------------------------------------------------------------------------------
>
>                 Key: HDFS-8038
>                 URL: https://issues.apache.org/jira/browse/HDFS-8038
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: tools
>            Reporter: Xiaoyu Yao
>            Assignee: Xiaoyu Yao
>            Priority: Minor
>             Fix For: 2.7.0
>
>         Attachments: HDFS-8038.00.patch, HDFS-8038.01.patch, 
> HDFS-8038.02.patch
>
>
> PBImageDelimitedTextWriter#getEntry taking the HDFS path and passing it 
> through java.io.File, which causes platform-specific behavior as the actual 
> results shown in TestOfflineImageViewer#testPBDelimitedWriter() on Windows OS.
> {code}
> expected:<[/emptydir, /dir0, /dir1/file2, /dir1, /dir1/file3, /dir2/file3, 
> /dir1/file0, /dir1/file1, /dir2/file1, /dir2/file2, /dir2, /dir0/file0, 
> /dir2/file0, /dir0/file1, /dir0/file2, /dir0/file3, /xattr]> 
> but was:<[\dir0, \dir0\file3, \dir0\file2, \dir0\file1, \xattr, \emptydir, 
> \dir0\file0, \dir1\file1, \dir1\file0, \dir1\file3, \dir1\file2, \dir2\file3, 
> \, \dir1, \dir2\file0, \dirContainingInvalidXMLChar&#0;here, \dir2, 
> \dir2\file2, \dir2\file1]>
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)