hadoop-mapreduce-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jason Lowe (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (MAPREDUCE-4980) Parallel test execution of hadoop-mapreduce-client-core
Date Thu, 14 Sep 2017 21:33:00 GMT

    [ 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

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

View raw message