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 [email protected] or file a JIRA ticket
with INFRA.
---
