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

Jason Lowe commented on MAPREDUCE-4980:
---------------------------------------

bq. Would folks be amenable if we commit a patch that basically 
fixes/modernizes the unit test code but doesn't actually enable parallel tests 
by default? 

Yes, that sounds great!  I'd hate for perfect to be the enemy of good here.  
There's some lingering issues with the parallel stuff, but in general I think 
there's great stuff here as far as making them better unit tests.

My apologies for not having had a chance to get back to this for a while, but 
I'd like to take a closer look at the patch tomorrow.  Just had a chance to 
glance at the patch so far, but one concern is that the patch is a lot bigger 
than it needs to be.  A lot of the changes are essentially whitespace and 
indentation reformats, and those changes aren't necessary to fix the unit test 
behaviors.  Do we really need those changes as part of this patch?

> Parallel test execution of hadoop-mapreduce-client-core
> -------------------------------------------------------
>
>                 Key: MAPREDUCE-4980
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4980
>             Project: Hadoop Map/Reduce
>          Issue Type: Test
>          Components: test
>    Affects Versions: 3.0.0-alpha1
>            Reporter: Tsuyoshi Ozawa
>            Assignee: Andrey Klochkov
>         Attachments: MAPREDUCE-4980.010.patch, MAPREDUCE-4980.011.patch, 
> MAPREDUCE-4980.012.patch, MAPREDUCE-4980.013.patch, MAPREDUCE-4980.014.patch, 
> MAPREDUCE-4980.015.patch, MAPREDUCE-4980.1.patch, MAPREDUCE-4980--n3.patch, 
> MAPREDUCE-4980--n4.patch, MAPREDUCE-4980--n5.patch, MAPREDUCE-4980--n6.patch, 
> MAPREDUCE-4980--n7.patch, MAPREDUCE-4980--n7.patch, MAPREDUCE-4980--n8.patch, 
> MAPREDUCE-4980.patch
>
>
> The maven surefire plugin supports parallel testing feature. By using it, the 
> tests can be run more faster.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

Reply via email to