[jira] [Commented] (DRILL-2359) ClassPathFileSystem.getFileStatus() should throw FileNotFoundException when path doesn't exist

["Deneche A. Hakim (JIRA)" <ji...@apache.org>]

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

Deneche A. Hakim commented on DRILL-2359:
-----------------------------------------

[~sphillips] let me know if the patch loogs good and I will add a unit test to it

> ClassPathFileSystem.getFileStatus() should throw FileNotFoundException when path doesn't exist
> ----------------------------------------------------------------------------------------------
>
>                 Key: DRILL-2359
>                 URL: https://issues.apache.org/jira/browse/DRILL-2359
>             Project: Apache Drill
>          Issue Type: Bug
>    Affects Versions: 0.8.0
>            Reporter: Deneche A. Hakim
>         Attachments: DRILL-2359.1.patch.txt
>
>
> {{ClassPathFileSystem.getFileStatus(Path args0)}} throws {{IOException}} when the path doesn't exist. It should throw {{FileNotFoundException}} instead. One particular method that fails to work correctly because of this is {{org.apache.hadoop.FileSystem.exists(Path f)}}:
> {code}
> public boolean exists(Path f) throws IOException {
>     try {
>       return getFileStatus(f) != null;
>     } catch (FileNotFoundException e) {
>       return false;
>     }
>   }
> {code}



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