Uploaded image for project: 'Spring Data Redis'
  1. Spring Data Redis
  2. DATAREDIS-71

upgrade jedis to 2.0.0 and support redis.clients.jedis.ShardedJedisPool for sharding

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: 1.0.0.M3
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      currenttly JedisConnectionFactory doesn't support sharding actually,sharding need a list of JedisShardInfo not a single JedisShardInfo.

      add:
      public void setShards(List<JedisShardInfo> shards);
      public void setHashingAlgo(Hashing hashingAlgo);

      jedisPool = new ShardedJedisPool(poolConfig,shards,hashingAlgo);
      jedis = new ShardedJedis(shards,hashingAlgo);

      remove:
      public void setShardInfo(JedisShardInfo shardInfo);

        Issue Links

          Activity

          Show
          quaff zhouyanming added a comment - FYI: https://github.com/xetorthio/jedis/issues/152
          Hide
          leonli Leon Li added a comment -

          Hi Costin,

          May I know when will the issue be implemented?
          And will the spring 3.1 annotation abstraction API supports the Jedis sharing tag? If so, how? can u please list some examples? Thx.

          Regards,

          Leon

          Show
          leonli Leon Li added a comment - Hi Costin, May I know when will the issue be implemented? And will the spring 3.1 annotation abstraction API supports the Jedis sharing tag? If so, how? can u please list some examples? Thx. Regards, Leon
          Hide
          costin Costin Leau added a comment -

          Unfortunately only one client/driver supports sharding which by itself is a work in-progress (as it ties into clustering). That's why we haven't started working on it and why we don't have an ETA.

          P.S. Ideally, sharding should really happen inside redis - there's no reason why not doing it plus it would be more transparent and fast then if it's done on the client side.

          Show
          costin Costin Leau added a comment - Unfortunately only one client/driver supports sharding which by itself is a work in-progress (as it ties into clustering). That's why we haven't started working on it and why we don't have an ETA. P.S. Ideally, sharding should really happen inside redis - there's no reason why not doing it plus it would be more transparent and fast then if it's done on the client side.
          Hide
          adrian0cg Adrian Hass added a comment -

          I second that, since it is where redis is going (and also similarly mongos as well):
          http://redis.io/topics/partitioning
          and Twemproxy seems good enough to do the job...

          Show
          adrian0cg Adrian Hass added a comment - I second that, since it is where redis is going (and also similarly mongos as well): http://redis.io/topics/partitioning and Twemproxy seems good enough to do the job...
          Hide
          mp911de Mark Paluch added a comment -

          Jedis was upgraded since quite a while to a newer version. Using Redis Cluster for sharding, see DATAREDIS-315.

          Show
          mp911de Mark Paluch added a comment - Jedis was upgraded since quite a while to a newer version. Using Redis Cluster for sharding, see DATAREDIS-315 .

            People

            • Assignee:
              thomasd Thomas Darimont
              Reporter:
              quaff zhouyanming
              Last updater:
              Mark Paluch
            • Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: