[jira] Commented: (HDFS-995) Replace usage of FileStatus#isDir()

[Eli Collins (JIRA)]

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

Eli Collins commented on HDFS-995:
----------------------------------

Places Sanjay called out in HDFS-245 that need to be fixed in this jira:

* FSImage#loadFilesUnderConstruction
{code}
        if (old.isDirectory()) ...
         add an additional check for isSymlink() -- or better !isFile()
{code}

* FSNamesystem#startFileInternal
{code}
} else if (myFile,isDirectory() {
         ....
} else if (myFile.isSymlink() {
           assert(false, cannot reach here ) <-- add
}
{code}

> Replace usage of FileStatus#isDir()
> -----------------------------------
>
>                 Key: HDFS-995
>                 URL: https://issues.apache.org/jira/browse/HDFS-995
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: name-node
>    Affects Versions: 0.22.0
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>             Fix For: 0.22.0
>
>
> HADOOP-6585 is going to deprecate FileStatus#isDir(). This jira is for 
> replacing all uses of isDir() in HDFS with checks of isDirectory(), isFile(), 
> or isSymlink() as needed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.