Re: GeodeRedisAdapter improvments/feedback

2017-02-15 Thread Jason Huynh
gt; > top 10 tweets which are trending then probably you want use > > partition-region for "sorted-set". > > > > > > From: Jason Huynh <jhu...@pivotal.io<mailto:jhu...@pivotal.io>> > > To: d...@geode.apache.org<mailto:d...@geode.apache.org&g

Re: GeodeRedisAdapter improvments/feedback

2017-02-15 Thread Real Wes
t; <user@geode.apache.org<mailto:user@geode.apache.org>>; > Hitesh Khamesra <hitesh...@yahoo.com<mailto:hitesh...@yahoo.com>> > Sent: Tuesday, February 14, 2017 3:15 PM > Subject: Re: GeodeRedisAdapter improvments/feedback > > Hi Hitesh, > > Not sure

Re: GeodeRedisAdapter improvments/feedback

2017-02-15 Thread Hitesh Khamesra
>>>The Redis adapter was designed so that we can scale all the Redis data structures horizontally. If you bring the data structures to region entry level, there is no reason for anyone to use our implementation over Redis. hmm, here we need to understand when we need to create partition-region for

Re: GeodeRedisAdapter improvments/feedback

2017-02-14 Thread William Markito Oliveira
Definitely not by asking in the middle of on-going discussion threads. Instructions: http://bfy.tw/A5ub :) Sent from my iPhone > On Feb 14, 2017, at 6:38 PM, Michael Vos wrote: > > How do I unsubscribe from this email list? > > Thank you, > > Michael Vos > Strategic

Re: GeodeRedisAdapter improvments/feedback

2017-02-14 Thread Michael Vos
How do I unsubscribe from this email list? Thank you, Michael Vos Strategic Partnerships 310-804-7223 | m...@pivotal.io > On Feb 14, 2017, at 3:37 PM, Dan Smith wrote: > > I also had a hard time reading this. It sounds like the tl;dr is that your > are proposing

Re: GeodeRedisAdapter improvments/feedback

2017-02-14 Thread Dan Smith
I also had a hard time reading this. It sounds like the tl;dr is that your are proposing storing each redis collection in a single region entry, rather than a a partition region? I guess the question is whether users will have a few very large collections that need to be partitioned, or lots and

Re: GeodeRedisAdapter improvments/feedback

2017-02-14 Thread Swapnil Bawaskar
The Redis adapter was designed so that we can scale all the Redis data structures horizontally. If you bring the data structures to region entry level, there is no reason for anyone to use our implementation over Redis. On Tue, Feb 14, 2017 at 3:15 PM Jason Huynh wrote: > Hi

Re: GeodeRedisAdapter improvments/feedback

2017-02-14 Thread Jason Huynh
Hi Hitesh, Not sure about everyone else, but I had a hard time reading this, however I think I figured out what you were describing... the only part I still am unsure about is Feedback/vote: both behaviour is desirable. Do you mean you want feedback and voting on whether both behaviors are

Re: GeodeRedisAdapter improvments/feedback

2017-02-14 Thread Real Wes
In what format do you want the feedback Hitesh? For now I’ll just comment: 1. Redis Type String No comments except that a future Geode value-add would be to extend the Jedis client so that the K/V’s are not compressed. In this way OQL and CQ will work. The tradeoff of this is that the data

GeodeRedisAdapter improvments/feedback

2017-02-14 Thread Hitesh Khamesra
Current GeodeRedisAdapter implementation is based on https://cwiki.apache.org/confluence/display/GEODE/Geode+Redis+Adapter+Proposal. We are looking for some feedback on Redis commands and their mapping to geode region. 1. Redis Type String   a. Usage Set k1 v1   b. Current implementation