Andrey,
I found a problem in INCR/DECR implementation.I think I misunderstood how it
works in the current memcached/rest implementation of Ignite, but I modified
Redis INCR/DECR now to behave the same way as memcached/rest.
Can you please have a look at https://issues.apache.org/jira/browse/IGNITE-4244
?
-Roman
On Wednesday, November 16, 2016 8:09 PM, Andrey Novikov
<anovi...@apache.org> wrote:
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
>
>
>
