Mailing-List: contact common-dev-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: common-dev@hadoop.apache.org
Content-Type: text/plain; charset=windows-1252
Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.6\))
Subject: Re: Status: hadoop-common precommit
From: Allen Wittenauer <aw@altiscale.com>
In-Reply-To: 
 <CAGB5D2arZ_Pb_=nAVUhJ+3uCD=bHD2UDp6yCKX1H48XN1r1LBg@mail.gmail.com>
Date: Mon, 19 Oct 2015 11:24:44 -0700
Content-Transfer-Encoding: quoted-printable
Message-Id: <EC9768CD-2EA4-4EB2-896B-9EDB0AFE22E9@altiscale.com>
References: <F827089A-B8A1-4662-86F8-AAB92FBB843A@altiscale.com>
 <C8AF58A8-4790-4FF6-83B0-BE6E1B1930BB@altiscale.com>
 <396078FB-C091-456B-9D5F-FC271F46914C@hortonworks.com>
 <56022842-FB6D-4A8A-9236-FB950A1DD9C4@altiscale.com>
 <CAGB5D2YHPwS1uZ8iODONZd2CrGqF_9f5F2WuGp-8ozMPe-iMag@mail.gmail.com>
 <88B588D6-8301-49BE-83A8-54E92A729988@altiscale.com>
 <CAGB5D2arZ_Pb_=nAVUhJ+3uCD=bHD2UDp6yCKX1H48XN1r1LBg@mail.gmail.com>
To: common-dev@hadoop.apache.org


On Oct 19, 2015, at 11:09 AM, Andrew Wang <andrew.wang@cloudera.com> =
wrote:

> Could we make it so we run RAT after building docs but before running
> tests? I think that's what's causing these issues, it looks like temp
> output from running tests. This would be a nice intermediate step =
toward
> that plug-in you mentioned, since test output is certainly not part of =
the
> release tarball.

	There=92s no real guarantee of the order a dev or worse the RE =
might run the build in, however.  The current order tests the worst =
possible case, and as a result discovered these bugs in the build that =
have been there for a very long time.  So moving the test to hide bad =
results sound counterproductive to me.  (It also might point to =
potential issues with parallel tests, but that=92s very case-by-case =
dependent of course.)

	Besides, with JDK8 now being tested against, these are likely =
the easier problems to fix given how broken javadoc is=85 ;)