gearpump-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From karol-brejna-i <...@git.apache.org>
Subject [GitHub] incubator-gearpump issue #11: fix #106 Gearpump Redis Integration
Date Wed, 22 Jun 2016 18:09:49 GMT
Github user karol-brejna-i commented on the issue:

    https://github.com/apache/incubator-gearpump/pull/11
  
    @darionyaphet 
    
    > Support more command such as : time to live , exist and delete .
    This is the reason I tend to agree @manuzhang. Let's have the "basic" source/sink. Let's
make RedisStorage a helper class (not a DataSink). This will let us release a functional version
of the connector and improve later.
    
    > Support Redis Cluster and Redis Sharding .
    Looks like Jedis has cluster support already (https://github.com/xetorthio/jedis#jedis-cluster).
Sharding, too. So it would be a matter of using the features. Again, I would try to incorporate
the features in next releases.
    
    > Add some Test Case
    As Manu wrote, test are not the blocker for an experiment module.
    
    My message is: let's release first version of Redis Connector soon and give ourselves
time to improve ;-)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message