[ 
https://issues.apache.org/jira/browse/HDFS-521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12770224#action_12770224
 ] 

Konstantin Boudnik commented on HDFS-521:
-----------------------------------------

I do respectfully disagree that we need to change all of it for a simple reason 
of using AspectJ both for fault- and injection purposes. If I've understood 
your proposal correctly, you want to have a separate set of everything for 
fault-injection and a 'normal injection' (for a lack of better word). right? I 
think it's too excessive to have 2 different sets of instrumented code.

I agree that 'fault-injection' isn't a best name for what the framework is 
clearly involving into... So we might consider this change later on. I think 
it's simpler and more appropriate to split non-fault-injecting and 
fault-injecting tests into different classes and name test methods according to 
the test plan. 

> Create new tests for pipeline
> -----------------------------
>
>                 Key: HDFS-521
>                 URL: https://issues.apache.org/jira/browse/HDFS-521
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: test
>    Affects Versions: 0.21.0, 0.22.0
>            Reporter: Konstantin Boudnik
>            Assignee: Konstantin Boudnik
>         Attachments: HDFS-521.patch, HDFS-521.patch, HDFS-521.patch, 
> HDFS-521.patch, HDFS-521.patch, HDFS-521.patch, HDFS-521.patch, 
> HDFS-521.patch, hdfs-521.patch, hdfs-521.patch, hdfs-521.patch, 
> HDFS-521.patch, HDFS-521.patch, HDFS-521.patch
>
>
> This JIRA is created to be a place holder for the test implemented as a part 
> of HDFS-483 fix

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