Dmitriy, At this moment supported command can be found in ticket description: https://issues.apache.org/jira/browse/IGNITE-3066 I think we shout add section for REDIS support on apacheignite.readme.io like for memcache Create ticket for documentation: https://issues.apache.org/jira/browse/IGNITE-4233
On Wed, Nov 16, 2016 at 6:12 PM, Dmitriy Setrakyan <dsetrak...@apache.org> wrote: > Roman, thanks for the contribution! Is there a documentation where I can > read about the supported Redis commands? > > On Wed, Nov 16, 2016 at 12: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 > > > > > > > > > > > > > > > > > > > > > > > >
