Roman, I reviewed your PR. Changed it a little and merged to master. Thanks for your contribution!
On Wed, Nov 16, 2016 at 9:39 AM, Roman Shtykh <rsht...@yahoo.com.invalid> wrote: > Andrey, > Sure, but I couldn't sign up -- "Cannot complete request due to license > limitations."Let's use github for now. > Thank you for your review! > -Roman > > > On Tuesday, November 15, 2016 6:12 PM, Andrey Novikov < > anovi...@apache.org> wrote: > > > Roman, > > I reviewed your code and added comments in JIRA. > > May we will try to use Upsource (http://reviews.ignite.apache.org/) for > code > review? > > > On Tue, Nov 15, 2016 at 1:22 PM, Roman Shtykh <rsht...@yahoo.com.invalid> > wrote: > > > Alexey, > > Thank you for your thorough reviews! I fixed the issues. > > -Roman > > > > > > On Tuesday, November 15, 2016 12:32 PM, Alexey Kuznetsov < > > akuznet...@apache.org> wrote: > > > > > > Roman, > > > > I reviewed your code and now it looks good for me. > > But I added two minor comments in JIRA. > > > > Also I think Andrey Novikov should take a look, as he has some experience > > in ignite-rest module. > > > > Andrey, take a look: > > > > Issue: https://issues.apache.org/jira/browse/IGNITE-3066 > > PR: https://github.com/apache/ignite/pull/1212 > > > > > > On Tue, Nov 15, 2016 at 9:27 AM, Roman Shtykh <rsht...@yahoo.com.invalid > > > > wrote: > > > > > Alexey, > > > Thank you!I answered and pushed the changes. > > > -Roman > > > > > > > > > On Tuesday, November 15, 2016 12:14 AM, Alexey Kuznetsov < > > > akuznet...@apache.org> wrote: > > > > > > > > > Roman, > > > > > > I made one more review, see my comments in JIRA issue. > > > > > > On Mon, Nov 7, 2016 at 1:30 PM, Alexey Kuznetsov < > akuznet...@apache.org> > > > wrote: > > > > > > > I will take a look on PR today. > > > > > > > > On Mon, Nov 7, 2016 at 11:35 AM, Roman Shtykh > > <rsht...@yahoo.com.invalid > > > > > > > > wrote: > > > > > > > >> Denis, > > > >> It is https://github.com/apache/ignite/pull/1212 > > > >> > > > >> Thank you, > > > >> Roman > > > >> > > > >> > > > >> On Saturday, November 5, 2016 4:56 AM, Denis Magda < > > > >> dma...@gridgain.com> wrote: > > > >> > > > >> > > > >> Roman, > > > >> > > > >> Would you mind making a pull-request? It’s not clear and easy to > > review > > > >> using the branch you provided > > > >> https://github.com/apache/ignite/tree/ignite-2788 < > > > >> https://github.com/apache/ignite/tree/ignite-2788> > > > >> > > > >> This link provides details how to achieve this > > > >> https://cwiki.apache.org/confluence/display/IGNITE/How+to+ > > > >> Contribute#HowtoContribute-1.CreateGitHubpull-request < > > > >> https://cwiki.apache.org/confluence/display/IGNITE/How+to+ > > > >> Contribute#HowtoContribute-1.CreateGitHubpull-request> > > > >> > > > >> Let us know if you have any issue preparing the pull-request. > > > >> > > > >> — > > > >> Denis > > > >> > > > >> > On Nov 3, 2016, at 6:24 PM, Roman Shtykh > <rsht...@yahoo.com.INVALID > > > > > > >> wrote: > > > >> > > > > >> > Igniters, > > > >> > Please review the issue.https://issues.apache.or > > > >> g/jira/browse/IGNITE-3066 > > > >> > > > > >> > Thank you,Roman > > > >> > > > >> > > > >> > > > >> > > > > > > > > > > > > > > > > -- > > > > Alexey Kuznetsov > > > > > > > > > > > > > > > > -- > > > Alexey Kuznetsov > > > > > > > > > > > > > > > > > -- > > Alexey Kuznetsov > > > > > > > > >