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

Íñigo Goiri commented on HADOOP-15106:
--------------------------------------

The doc change looks good.
Is there any unit test covering this?
I remember that the positive cases were tested in some class (which one was 
it?), can we add an assert for this exception?

> FileSystem::open(PathHandle) should throw a specific exception on validation 
> failure
> ------------------------------------------------------------------------------------
>
>                 Key: HADOOP-15106
>                 URL: https://issues.apache.org/jira/browse/HADOOP-15106
>             Project: Hadoop Common
>          Issue Type: Improvement
>            Reporter: Chris Douglas
>            Priority: Minor
>         Attachments: HADOOP-15106.00.patch, HADOOP-15106.01.patch, 
> HADOOP-15106.02.patch, HADOOP-15106.03.patch
>
>
> Callers of {{FileSystem::open(PathHandle)}} cannot distinguish between I/O 
> errors and an invalid handle. The signature should include a specific, 
> checked exception for this case.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to