[ https://issues.apache.org/jira/browse/HDFS-6006?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Akira AJISAKA updated HDFS-6006: -------------------------------- Attachment: HDFS-6006.patch Attaching a simple patch. > Remove duplicating code in FSNameSystem#getFileInfo > --------------------------------------------------- > > Key: HDFS-6006 > URL: https://issues.apache.org/jira/browse/HDFS-6006 > Project: Hadoop HDFS > Issue Type: Improvement > Affects Versions: 2.3.0 > Reporter: Akira AJISAKA > Assignee: Akira AJISAKA > Priority: Trivial > Labels: newbie > Attachments: HDFS-6006.patch > > > In FSNameSystem#getFileInfo, it checks src file name two times. > {code} > if (!DFSUtil.isValidName(src)) { > throw new InvalidPathException("Invalid file name: " + src); > } > HdfsFileStatus stat = null; > FSPermissionChecker pc = getPermissionChecker(); > checkOperation(OperationCategory.READ); > if (!DFSUtil.isValidName(src)) { > throw new InvalidPathException("Invalid file name: " + src); > } > {code} > The latter should be removed. -- This message was sent by Atlassian JIRA (v6.1.5#6160)