Creating a review request with "drill-git" Group should be usually enough. However, it would not hurt to send a reminder mail to the drill-dev list so that it get some additional attention.
I'll review the patch later today. aditya... On Wed, Oct 22, 2014 at 7:34 AM, Jim Scott <[email protected]> wrote: > I created this review a month ago now. What is the process to get it > reviewed? > https://reviews.apache.org/r/25949/ > > On Tue, Sep 23, 2014 at 2:24 PM, Yash Sharma <[email protected]> wrote: > > > Hi Jim, > > We use the ReviewBoard[1] for patch reviews. > > Once we have it reviewed (implementing review comments and getting +1's) > we > > transit to the Patch Available & assign it to Drill Committer. > > > > 1. https://reviews.apache.org > > > > Thanks > > > > On Wed, Sep 24, 2014 at 12:48 AM, Jim Scott <[email protected]> wrote: > > > > > Is there a process document for submitting patches, etc? > > > > > > I have attached a patch to DRILL-1440. Not sure on properly tagging the > > > JIRA, or if I should reassign it once a patch is available, etc.. > > > > > > On Mon, Sep 22, 2014 at 11:55 PM, Jacques Nadeau <[email protected]> > > > wrote: > > > > > > > I've added you as a contributor on JIRA so please assign these to > > > > yourself. Looking forward to the patches! > > > > > > > > On Mon, Sep 22, 2014 at 9:24 AM, Jim Scott <[email protected]> > > wrote: > > > > > > > > > I started working on these: > > > > > https://issues.apache.org/jira/browse/DRILL-1318 > > > > > https://issues.apache.org/jira/browse/DRILL-1440 > > > > > > > > > > I am almost complete (I think), at least close to having others > > review > > > > for > > > > > a patch to be submitted. > > > > > > > > > > > > > > > > > > > > > -- > > > *Jim Scott* > > > Director, Enterprise Strategy & Architecture > > > +1 (347) 746-9281 > > > > > > <http://www.mapr.com/> > > > [image: MapR Technologies] <http://www.mapr.com> > > > > > > > > > -- > *Jim Scott* > Director, Enterprise Strategy & Architecture > +1 (347) 746-9281 > > <http://www.mapr.com/> > [image: MapR Technologies] <http://www.mapr.com> >
