You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@drill.apache.org by "Ramana Inukonda Nagaraj (JIRA)" <ji...@apache.org> on 2015/04/22 23:36:59 UTC

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

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

Ramana Inukonda Nagaraj commented on DRILL-2359:
------------------------------------------------

Hakim,

Could you please add a test and close. 

> 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
>          Components: Execution - Flow
>    Affects Versions: 0.8.0
>            Reporter: Deneche A. Hakim
>            Assignee: Deneche A. Hakim
>            Priority: Blocker
>             Fix For: 0.8.0
>
>         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)