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

Konstantin Boudnik commented on MAPREDUCE-670:
----------------------------------------------

It seems that indentation isn't consistent across the changes.
{code}
        <fileset dir="${test.src.dir}/mapred" 
excludes="**/${test.exclude}.java">
          <patternset>
          <includesfile name="@{test.file}"/>
          </patternset>
          </fileset>
{code}

or

{code}
  <target name="run-test-mapred" depends="compile-mapred-test" description="Run 
mapred unit tests">
     <macro-test-runner test.file="${test.mapred.all.tests.file}" />
  </target> 
{code}

Looks good otherwise

>  Create target for 10 minute patch test build for mapreduce
> -----------------------------------------------------------
>
>                 Key: MAPREDUCE-670
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-670
>             Project: Hadoop Map/Reduce
>          Issue Type: Test
>          Components: build
>            Reporter: Jothi Padmanabhan
>            Assignee: Jothi Padmanabhan
>         Attachments: FastTestsInfo.xls, mapred-670.patch
>
>
> Creating a new Jira to track HADOOP-5628 for MapReduce

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