Dmitriy,
I think additions of some operations on sets and hashes are pretty
straightforward.
But I think we need to extend REST implementation to cover even more Redis
commands (it won't probably remove all limitations we have now though, like not
having expiration on keys), or create a Redis processor to access Ignite Cache
API directly (which will give us more freedom on how we implement Redis
commands). This is my understanding. I would appreciate more discussion and
advice on how to continue (probably in
https://issues.apache.org/jira/browse/IGNITE-2788)
Roman
On Friday, November 18, 2016 5:42 PM, Dmitriy Setrakyan
<[email protected]> wrote:
Thanks, Roman! I looked at the content and it looks OK.
Are we going to be adding more Redis commands? If yes, would be nice if we
would mention which commands we plan to add in the nearest future.
D.
On Fri, Nov 18, 2016 at 4:23 AM, Roman Shtykh <[email protected]> wrote:
Dmitriy,
I have contents prepared for the Redis category page on
https://issues.apache.org/ jira/browse/IGNITE-4233Later I will add examples for
Java and some other languages.
-Roman
On Thursday, November 17, 2016 9:59 AM, Roman Shtykh
<[email protected]> wrote:
Andrey, thank you!
Dmitriy, I will when we have 1.8.0 version on readme.io.
-Roman
On Wednesday, November 16, 2016 8:38 PM, Dmitriy Setrakyan
<[email protected]> wrote:
Roman, do you mind contributing the documentation for this feature?
On Wed, Nov 16, 2016 at 12:27 PM, Andrey Novikov <[email protected]>
wrote:
> 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 <[email protected]>
> 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 <[email protected]>
> > 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
> <[email protected]
> > >
> > > 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 <
> > > > [email protected]> 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
> > <[email protected]
> > > >
> > > > wrote:
> > > >
> > > > > Alexey,
> > > > > Thank you for your thorough reviews! I fixed the issues.
> > > > > -Roman
> > > > >
> > > > >
> > > > > On Tuesday, November 15, 2016 12:32 PM, Alexey Kuznetsov <
> > > > > [email protected]> 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
> > > <[email protected]
> > > > >
> > > > > wrote:
> > > > >
> > > > > > Alexey,
> > > > > > Thank you!I answered and pushed the changes.
> > > > > > -Roman
> > > > > >
> > > > > >
> > > > > > On Tuesday, November 15, 2016 12:14 AM, Alexey Kuznetsov <
> > > > > > [email protected]> wrote:
> > > > > >
> > > > > >
> > > > > > Roman,
> > > > > >
> > > > > > I made one more review, see my comments in JIRA issue.
> > > > > >
> > > > > > On Mon, Nov 7, 2016 at 1:30 PM, Alexey Kuznetsov <
> > > > [email protected]>
> > > > > > wrote:
> > > > > >
> > > > > > > I will take a look on PR today.
> > > > > > >
> > > > > > > On Mon, Nov 7, 2016 at 11:35 AM, Roman Shtykh
> > > > > <[email protected]
> > > > > > >
> > > > > > > wrote:
> > > > > > >
> > > > > > >> Denis,
> > > > > > >> It is https://github.com/apache/ ignite/pull/1212
> > > > > > >>
> > > > > > >> Thank you,
> > > > > > >> Roman
> > > > > > >>
> > > > > > >>
> > > > > > >> On Saturday, November 5, 2016 4:56 AM, Denis Magda <
> > > > > > >> [email protected]> 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
> > > > <[email protected]
> > > > > >
> > > > > > >> wrote:
> > > > > > >> >
> > > > > > >> > Igniters,
> > > > > > >> > Please review the issue.https://issues.apache.or
> > > > > > >> g/jira/browse/IGNITE-3066
> > > > > > >> >
> > > > > > >> > Thank you,Roman
> > > > > > >>
> > > > > > >>
> > > > > > >>
> > > > > > >>
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Alexey Kuznetsov
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Alexey Kuznetsov
> > > > > >
> > > > > >
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Alexey Kuznetsov
> > > > >
> > > > >
> > > > >
> > > >
> > > >
> > > >
> > >
> >
>
