Mailing-List: contact dev-help@mahout.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@mahout.apache.org
Received-SPF: pass (athena.apache.org: domain of andrew.musselman@gmail.com
 designates 74.125.82.53 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAOwb3gPDhmp+z07aixdRY3y=g2_43G419A_B0Gh=pWmxgGKPDg@mail.gmail.com>
References: 
 <CAOwb3gMYKQAn6s=FJ10vkyeDGmygf1cxJ6EjJdMOkskkU8ZTfg@mail.gmail.com>
	<5163FCD7.5050409@googlemail.com>
	<CAOwb3gPDhmp+z07aixdRY3y=g2_43G419A_B0Gh=pWmxgGKPDg@mail.gmail.com>
Date: Tue, 9 Apr 2013 06:16:06 -0700
Message-ID: 
 <CANg8BGCJ4qWRAZr3sY2nmiJfWtwVtvec1p3oCLdV_1nRE-XwAw@mail.gmail.com>
Subject: Re: Code reviews and reviewers
From: Andrew Musselman <andrew.musselman@gmail.com>
To: dev@mahout.apache.org
Content-Type: multipart/alternative; boundary=089e013c6ab025062a04d9ed5b9f

--089e013c6ab025062a04d9ed5b9f
Content-Type: text/plain; charset=ISO-8859-1

+1 for code reviews
+1 for Review Board
+1 for unit tests and integration tests


On Tue, Apr 9, 2013 at 5:03 AM, Dan Filimon <dangeorge.filimon@gmail.com>wrote:

> Thanks Sebastian!
>
> But let's talk about policy. Wouldn't everyone agree that more code reviews
> are a good thing?
> Ideally, everyone does it and becomes familiar with the code base.
>
> So, even though I, for instance wouldn't be able to say much about
> something I haven't worked on, like recommendation systems, a sanity check
> is always good to have.
> Also good for having a cohesive style and ensuring the code is easy to read
> and is adequately documented.
>
> And, best of all, I get the added bonus of having to read up (at least an
> overview) of how these systems work to provide some more meaningful
> feedback.
>
> Right now, it looks like there aren't that many changes going through
> ReviewBoard. Sure, there's JIRA, but for more significant changes?
>
> Any thoughts on what's best to do?
>
>
>
> On Tue, Apr 9, 2013 at 2:34 PM, Sebastian Schelter
> <ssc.open@googlemail.com>wrote:
>
> > Dan,
> >
> > it's a pleasure to review your code. Ask me anytime :)
> >
> > On 09.04.2013 13:31, Dan Filimon wrote:
> > > Hi everyone,
> > >
> > > Sebastian has been reviewing my code on ReviewBoard [1] for a while now
> > and
> > > I feel bad for always asking him to do it. :)
> > >
> > > Is there anyone else who could have a look (I'll also volunteer when
> you
> > > need a reviewer)?
> > >
> > > [1] https://reviews.apache.org/r/10372/
> > >
> >
> >
>

--089e013c6ab025062a04d9ed5b9f--