[
https://issues.apache.org/jira/browse/HADOOP-6730?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12862423#action_12862423
]
Ravi Phulari commented on HADOOP-6730:
--------------------------------------
FileContext.Util.Copy() operation was failing with file not found exception if
destination file does not exits , this was happening due to the fact that
getFileStatus was out of try block.
bq. The patch just has util classes - perhaps you forgot to add the test class
in your diff?
I have included test runner with util base. Please check that there are two
classes added with this patch - {{TestFcLocalFsUtil.java}} and
{{FileContextUtilBase.java}}
> Bug in FileContext#copy
> -----------------------
>
> Key: HADOOP-6730
> URL: https://issues.apache.org/jira/browse/HADOOP-6730
> Project: Hadoop Common
> Issue Type: Bug
> Components: fs
> Affects Versions: 0.22.0
> Reporter: Eli Collins
> Assignee: Ravi Phulari
> Fix For: 0.22.0
>
> Attachments: HADOOP-6730.patch
>
>
> Thanks to Eli, He noticed that there is no test for FileContext#Copy
> operation.
> On further investigation with the help of Sanjay we found that there is bug
> in FileContext#checkDest.
> {noformat}
> FileStatus dstFs = getFileStatus(dst);
> try {
> if (dstFs.isDir()) {
> if (null == srcNa
> {noformat}
> *FileStatus dstFs = getFileStatus(dst);* should be in try...catch block.
> {noformat}
> try {
> FileStatus dstFs = getFileStatus(dst);
> if (dstFs.isDir()) {
> if (null == srcNa
> {noformat}
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
