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

Jason Lowe commented on HADOOP-9287:
------------------------------------

bq. All changes in YARN code introduced in this patch are required because YARN 
test classes depend on classes from hadoop-common. If we extract these changes 
somewhere else, this patch would lead to a non-compilable code. 

That doesn't seem to be the case.  I extracted the changes under 
hadoop-yarn-project into a separate patch, applied just that patch to trunk, 
and trunk built just fine.

Even if that were the case, it's better to track the changes for YARN under a 
separate JIRA that is dependent upon this one being integrated.  The only 
reason we aren't doing the same for HDFS is that those changes are intertwined. 
 We can't commit just the common changes without breaking the HDFS build and 
vice-versa.  That is not the case with the YARN changes, as the patch builds 
just fine without them.  Since they are independent changes, they should be 
tracked in their respective projects.
                
> Parallel testing hadoop-common
> ------------------------------
>
>                 Key: HADOOP-9287
>                 URL: https://issues.apache.org/jira/browse/HADOOP-9287
>             Project: Hadoop Common
>          Issue Type: Test
>          Components: test
>    Affects Versions: 3.0.0
>            Reporter: Tsuyoshi OZAWA
>            Assignee: Andrey Klochkov
>         Attachments: HADOOP-9287.1.patch, HADOOP-9287--N3.patch, 
> HADOOP-9287--N3.patch, HADOOP-9287--N4.patch, HADOOP-9287--N5.patch, 
> HADOOP-9287.patch, HADOOP-9287.patch
>
>
> The maven surefire plugin supports parallel testing feature. By using it, the 
> tests can be run more faster.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to