hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hadoop QA (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-9819) FileSystem#rename is broken, deletes target when renaming link to itself
Date Fri, 19 Aug 2016 23:29:20 GMT

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

Hadoop QA commented on HADOOP-9819:
-----------------------------------

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m  0s{color} | {color:blue}
Docker mode activated. {color} |
| {color:red}-1{color} | {color:red} patch {color} | {color:red}  0m  6s{color} | {color:red}
HADOOP-9819 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute
for help. {color} |
\\
\\
|| Subsystem || Report/Notes ||
| JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12805639/HADOOP-9819.03.patch
|
| JIRA Issue | HADOOP-9819 |
| Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/10318/console |
| Powered by | Apache Yetus 0.4.0-SNAPSHOT   http://yetus.apache.org |


This message was automatically generated.



> FileSystem#rename is broken, deletes target when renaming link to itself
> ------------------------------------------------------------------------
>
>                 Key: HADOOP-9819
>                 URL: https://issues.apache.org/jira/browse/HADOOP-9819
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs
>    Affects Versions: 3.0.0-alpha1
>            Reporter: Arpit Agarwal
>            Assignee: Andras Bokor
>         Attachments: HADOOP-9819.01.patch, HADOOP-9819.02.patch, HADOOP-9819.03.patch
>
>
> Uncovered while fixing TestSymlinkLocalFsFileSystem on Windows.
> This block of code deletes the symlink, the correct behavior is to do nothing.
> {code:java}
>     try {
>       dstStatus = getFileLinkStatus(dst);
>     } catch (IOException e) {
>       dstStatus = null;
>     }
>     if (dstStatus != null) {
>       if (srcStatus.isDirectory() != dstStatus.isDirectory()) {
>         throw new IOException("Source " + src + " Destination " + dst
>             + " both should be either file or directory");
>       }
>       if (!overwrite) {
>         throw new FileAlreadyExistsException("rename destination " + dst
>             + " already exists.");
>       }
>       // Delete the destination that is a file or an empty directory
>       if (dstStatus.isDirectory()) {
>         FileStatus[] list = listStatus(dst);
>         if (list != null && list.length != 0) {
>           throw new IOException(
>               "rename cannot overwrite non empty destination directory " + dst);
>         }
>       }
>       delete(dst, false);
> {code}



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

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org


Mime
View raw message