[ 
https://issues.apache.org/jira/browse/MAPREDUCE-1084?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786545#action_12786545
 ] 

Konstantin Boudnik commented on MAPREDUCE-1084:
-----------------------------------------------

svn external patch is better to be abandoned because vanilla git doesn't 
understand svn:externals. So, lets consider the patch without externals as the 
only alternative at the moment.

Few comments:
- it seems like patch is missing {{src/test/aop/build/aop.xml}} file
- target name {{jar-mapred-test-fault-inject}} needs to be changed to 
{{jar-mapred-fault-inject}} to be compliant with Common and HDFS names


> Implementing aspects development and fault injeciton framework for MapReduce
> ----------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-1084
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1084
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>          Components: build, test
>            Reporter: Konstantin Boudnik
>            Assignee: Sreekanth Ramakrishnan
>         Attachments: mapreduce-1084-1-withoutsvnexternals.patch, 
> mapreduce-1084-1.patch
>
>
> Similar to HDFS-435 and HADOOP-6204 this JIRA will track the introduction of 
> injection framework for MapReduce.
> After HADOOP-6204 is in place this particular modification should be very 
> trivial and would take importing (via svn:external) of src/test/build and 
> some tweaking of the 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