[
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: [email protected]
For additional commands, e-mail: [email protected]
