Brock Noland 2012-12-07, 16:11
Jarek Jarcec Cecho 2012-12-07, 16:26
Great to hear you were thinking the same thing! I have found the
script they use to do the checking:
and I think we could probably implement something similar. I agree
that our build machines are much busier so I don't think we can use it
as a hard requirement for commit. The biggest open question in my mind
is how do they query JIRA for patches? Grab the correct patch off of
the JIRA? Schedule the job to poll for these things?
On Fri, Dec 7, 2012 at 10:26 AM, Jarek Jarcec Cecho <[EMAIL PROTECTED]> wrote:
> Hi Brock,
> that is an excellent idea and I'm definitely in favour of that. In fact I wanted to propose something similar myself.
> However I did not quite finished my own investigation yet. It seems that this pre commit hooks are done on projects that were mostly donated by Yahoo! (Hadoop, Oozie) on Jenkins nodes named "hadoopX". The hadoop named nodes have comment "Build machine for Hadoop project builds", so I'm not sure that they are available for other projects as well.
> With removing those 9 hadoop notes, it's like 10 other nodes in total for entire Apache organization and I do know that sometimes the build queue might be quite huge. Having said that it might be good idea to ask Infra team if pre-commit hooks are available to all projects or only to projects that have "special hardware" available.
> On Fri, Dec 07, 2012 at 10:11:53AM -0600, Brock Noland wrote:
>> I like how in the Hadoop project if you mark a patch "Patch Available"
>> jenkins applies the patch to trunk, runs the tests, and outputs the
>> result to the JIRA. Here is one example:
>> I think we should still run tests manually before commit. However,
>> this will often give a reviewer an indicator of whether the change
>> breaks a test without even having to download the code.
Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/
Jarek Jarcec Cecho 2012-12-07, 23:20
Mike Percy 2012-12-08, 08:39
Brock Noland 2012-12-20, 06:38
Jarek Jarcec Cecho 2012-12-20, 15:04
Brock Noland 2012-12-21, 00:25