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

Jakob Homan commented on HADOOP-6261:
-------------------------------------

Patch review:
  * {code}System.getProperty("test.build.data") + "/testContextURI";{code} 
needs a default option, see HADOOP-5916
  * Remove the TODO comment 
  * Local and S3 implementations, you're catching but not handling an 
IOException.  The tests shouldn't proceed if one is thrown; don't catch it.
  * FileContextURIBase.java:121 Says an IOException is expected but catches an 
IllegalArgumentException.

> Junit tests for FileContextURI
> ------------------------------
>
>                 Key: HADOOP-6261
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6261
>             Project: Hadoop Common
>          Issue Type: Test
>          Components: test
>            Reporter: Ravi Phulari
>            Assignee: Ravi Phulari
>         Attachments: HADOOP-6261.patch, HADOOP-6261.patch
>
>
> FileContextURI unit tests. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to