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

Philip Zeyliger commented on HADOOP-6003:
-----------------------------------------

It looks like some of the patches have inconsistent usage of tabs/spaces; you 
may want to clean that up.

bq. It is expected to see unit tests failing with faults in place. We might 
need to develop different kind of tests to utilize fault injection in a better 
way.

I think you're right on here.  It seems like having a global setting for 
injection probability will mostly lead to test results that are hard to grok.  
Have you thought about setting up a separate package/target for tests that are 
designed to interact with injected faults?  Once one or two of those tests 
exists, it'll be clearer whether "ant run-test-hdfs 
-Dfault.probability.FSDataset=3" is the right API.

> AspectJ framework for HDFS code and tests
> -----------------------------------------
>
>                 Key: HADOOP-6003
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6003
>             Project: Hadoop Core
>          Issue Type: Sub-task
>    Affects Versions: 0.20.0
>            Reporter: Konstantin Boudnik
>            Assignee: Konstantin Boudnik
>             Fix For: 0.21.0
>
>         Attachments: HADOOP-6003.patch, HADOOP-6003.sh
>
>
> This subtask takes care about HDFS part of Hadoop only. Others will be added 
> later as needed: it will include only new aspects development and 
> modifications of build.xml file

-- 
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