Atri, please, see my comments at the Jira ticket. -- Artem --
On Thu, Jun 4, 2015 at 7:44 PM, Atri Sharma <atri.j...@gmail.com> wrote: > Thanks. > > Sorry for the noise on this one. I am off next week so it's helpful if we > can close this by this week if possible. I am willing to work in my > midnight for that if needed :) > On 4 Jun 2015 01:53, "Dmitriy Setrakyan" <dsetrak...@apache.org> wrote: > > > Atri, > > > > Thanks for providing a patch to Ignite! I am sure someone in the > community > > will be looking into it shortly (if not already). > > > > Once your patch is accepted, you will be eligible to become a committer. > > > > D. > > > > On Tue, Jun 2, 2015 at 7:03 AM, Atri Sharma <atri.j...@gmail.com> wrote: > > > > > Patch version 1 is available. Please see and comment. > > > > > > On Mon, Jun 1, 2015 at 9:18 PM, Atri Sharma <atri.j...@gmail.com> > wrote: > > > > > > > Thanks. > > > > > > > > On Mon, Jun 1, 2015 at 9:16 PM, Yakov Zhdanov <yzhda...@apache.org> > > > wrote: > > > > > > > >> Did you see the comment in the ticket? Andrey Gura has put some tips > > > >> there. > > > >> You can ask further questions in the ticket. > > > >> > > > >> Good luck! > > > >> > > > >> --Yakov > > > >> > > > >> 2015-06-01 18:40 GMT+03:00 Atri Sharma <atri.j...@gmail.com>: > > > >> > > > >> > Thanks Yakov. > > > >> > > > > >> > Can you also please advice at code areas where I should look at > for > > > this > > > >> > please? > > > >> > > > > >> > On Mon, Jun 1, 2015 at 9:03 PM, Yakov Zhdanov < > yzhda...@apache.org> > > > >> wrote: > > > >> > > > > >> > > Arti you can read about sprint process here > > > >> > > > https://cwiki.apache.org/confluence/display/IGNITE/Sprint+Process > > > >> > > > > > >> > > This page will be soon updated with information on how to format > > and > > > >> > submit > > > >> > > patch and check CI tests. > > > >> > > > > > >> > > --Yakov > > > >> > > > > > >> > > 2015-06-01 12:54 GMT+03:00 Atri Sharma <atri.j...@gmail.com>: > > > >> > > > > > >> > > > I would like to work on 917, have assigned it to myself. > > > >> > > > > > > >> > > > Please advice on the approach and code areas this patch will > > need > > > >> > > > modifications in. > > > >> > > > > > > >> > > > -- > > > >> > > > Regards, > > > >> > > > > > > >> > > > Atri > > > >> > > > *l'apprenant* > > > >> > > > > > > >> > > > > > >> > > > > >> > > > > >> > > > > >> > -- > > > >> > Regards, > > > >> > > > > >> > Atri > > > >> > *l'apprenant* > > > >> > > > > >> > > > > > > > > > > > > > > > > -- > > > > Regards, > > > > > > > > Atri > > > > *l'apprenant* > > > > > > > > > > > > > > > > -- > > > Regards, > > > > > > Atri > > > *l'apprenant* > > > > > >
