ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Roman Shtykh <rsht...@yahoo.com.INVALID>
Subject Re: IGNITE-3066 Set of Redis commands that can be easily implemented via existing REST commands
Date Wed, 16 Nov 2016 02:39:45 GMT
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 <anovikov@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 <rshtykh@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 <
> akuznetsov@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 <rshtykh@yahoo.com.invalid>
> wrote:
>
> > Alexey,
> > Thank you!I answered and pushed the changes.
> > -Roman
> >
> >
> >    On Tuesday, November 15, 2016 12:14 AM, Alexey Kuznetsov <
> > akuznetsov@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 <akuznetsov@apache.org>
> > wrote:
> >
> > > I will take a look on PR today.
> > >
> > > On Mon, Nov 7, 2016 at 11:35 AM, Roman Shtykh
> <rshtykh@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 <
> > >> dmagda@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 <rshtykh@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
>
>
>

   
Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message