[ https://issues.apache.org/jira/browse/HADOOP-2634?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12560101#action_12560101 ]
Doug Cutting commented on HADOOP-2634: -------------------------------------- Hairong has addressed these inconsistencies in HADOOP-2566. Yes, the implementation of exists() in terms of getFileStatus() would be simple. However it is considered bad style to use exceptions for normal control flow, and exists() returning false is a normal condition. We might just have to live with that... > Deprecate exists() and isDir() to simplify ClientProtocol. > ---------------------------------------------------------- > > Key: HADOOP-2634 > URL: https://issues.apache.org/jira/browse/HADOOP-2634 > Project: Hadoop > Issue Type: Improvement > Components: dfs > Affects Versions: 0.15.0 > Reporter: Konstantin Shvachko > > ClientProtocol can be simplified by removing two methods > {code} > public boolean exists(String src) throws IOException; > public boolean isDir(String src) throws IOException; > {code} > This is a redundant api, which can be implemented in DFSClient as convenience > methods using > {code} > public DFSFileInfo getFileInfo(String src) throws IOException; > {code} > Note that we already deprecated several Filesystem method and advised to use > getFileStatus() instead. > Should we deprecate them in 0.16? -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.