Re: [DISCUSS] Moving redis to a separate module

2019-03-12 Thread Anilkumar Gingade
+1 On Tue, Mar 12, 2019 at 5:32 PM John Blum wrote: > Definitely a reasonable change. Perhaps, for consistency sake, the same > should be applied to Geode's Memcached support? (in another PR). > > > On Tue, Mar 12, 2019 at 4:23 PM Dan Smith wrote: > > > I created a PR to move our redis support

Re: [DISCUSS] Moving redis to a separate module

2019-03-12 Thread John Blum
Definitely a reasonable change. Perhaps, for consistency sake, the same should be applied to Geode's Memcached support? (in another PR). On Tue, Mar 12, 2019 at 4:23 PM Dan Smith wrote: > I created a PR to move our redis support to a separate module. Let me know > what you think: > > https://g

[DISCUSS] Moving redis to a separate module

2019-03-12 Thread Dan Smith
I created a PR to move our redis support to a separate module. Let me know what you think: https://github.com/apache/geode/pull/3284 Geode servers will still include redis on the classpath, so the only effect of this is that if you are launching a server based on the maven dependencies, you will