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

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

- The toString() below should be removed.
{code}
+    ConfigUtil.addLink(conf, "/tmp", new URI(fHdfs.getUri().toString() + 
"/tmp"));
{code}
Also, how about use different names for "/tmp", say "/hdfstmp" and "/viewfstmp"?

- Instead of checking null, how about get a file checksum from 
DistributedFileSystem for the same file and then check equality?
                
> Add a test to verify getFileChecksum works with ViewFS
> ------------------------------------------------------
>
>                 Key: HDFS-2505
>                 URL: https://issues.apache.org/jira/browse/HDFS-2505
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: test
>    Affects Versions: 0.23.0
>            Reporter: Ravi Prakash
>            Assignee: Ravi Prakash
>              Labels: test
>             Fix For: 0.24.0
>
>         Attachments: HDFS-2505.patch
>
>
> Please refer to HADOOP-7770. getFileChecksum was failing on files such as 
> /tmp/someFile, but working fine for /someDir/someFile.
> The fix is in HADOOP-7770 but we should have a test in HDFS which checks this 
> functionality (this test will fail until HADOOP-7770 is checked in)

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