Mailing-List: contact yarn-dev-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: yarn-dev@hadoop.apache.org
Received-SPF: neutral (athena.apache.org: local policy)
Subject: Re: Commits breaking compilation of MR 'classic' tests
Mime-Version: 1.0 (Apple Message framework v1084)
Content-Type: multipart/alternative; boundary=Apple-Mail-924-1006280588
From: Arun C Murthy <acm@hortonworks.com>
In-Reply-To: <CC889CDB.9247%evans@yahoo-inc.com>
Date: Wed, 26 Sep 2012 10:31:07 -0700
Cc: common-dev@hadoop.apache.org,
 yarn-dev@hadoop.apache.org,
 mapreduce-dev@hadoop.apache.org
Message-Id: <A8043143-CAEE-4C11-A7DF-94061F58E62F@hortonworks.com>
References: <CC889CDB.9247%evans@yahoo-inc.com>
To: hdfs-dev@hadoop.apache.org,
 Robert Evans <evans@yahoo-inc.com>

--Apple-Mail-924-1006280588
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain;
	charset=us-ascii

Fair, however there are still tests which need to be ported over. We can =
remove them after the port.

On Sep 26, 2012, at 9:54 AM, Robert Evans wrote:

> As per my comment on the bug.  I though we were going to remove them.
>=20
> MAPREDUCE-4266 only needs a little bit more work, change a patch to a
> script, before they disappear entirely.  I would much rather see dead =
code
> die then be maintained for a few tests that are mostly testing the =
dead
> code itself.
>=20
>=20
> --Bobby
>=20
> On 9/26/12 9:39 AM, "Arun C Murthy" <acm@hortonworks.com> wrote:
>=20
>> Point. I've opened =
https://issues.apache.org/jira/browse/MAPREDUCE-4687
>> to track this.
>>=20
>> On Sep 25, 2012, at 9:33 PM, Eli Collins wrote:
>>=20
>>> How about adding this step to the MR PreCommit jenkins job so it's =
run
>>> as part test-patch?
>>>=20
>>> On Tue, Sep 25, 2012 at 7:48 PM, Arun C Murthy <acm@hortonworks.com>
>>> wrote:
>>>> Committers,
>>>>=20
>>>> As most people are aware, the MapReduce 'classic' tests (in
>>>> hadoop-mapreduce-project/src/test) still need to built using ant =
since
>>>> they aren't mavenized yet.
>>>>=20
>>>> I've seen several commits (and 2 within the last hour i.e.
>>>> MAPREDUCE-3681 and MAPREDUCE-3682) which lead me to believe
>>>> developers/committers aren't checking for this.
>>>>=20
>>>> Henceforth, with all changes, before committing, please do run:
>>>> $ mvn install
>>>> $ cd hadoop-mapreduce-project
>>>> $ ant veryclean all-jars -Dresolvers=3Dinternal
>>>>=20
>>>> These instructions were already in
>>>> http://wiki.apache.org/hadoop/HowToReleasePostMavenization and I've
>>>> just updated http://wiki.apache.org/hadoop/HowToContribute.
>>>>=20
>>>> thanks,
>>>> Arun
>>>>=20
>>=20
>> --
>> Arun C. Murthy
>> Hortonworks Inc.
>> http://hortonworks.com/
>>=20
>>=20
>=20

--
Arun C. Murthy
Hortonworks Inc.
http://hortonworks.com/


--Apple-Mail-924-1006280588--