Support cluster connection http://redis.io/topics/cluster-tutorial

Both Predis and phpredis connectors now support it seamlessly.

CommentFileSizeAuthor
#13 support-cluster-connections-2556097-13.patch21.6 KBgerson.analista
#3 redis_cluster_variables.patch3.12 KBducktape

Comments

pounard created an issue. See original summary.

pounard’s picture

pounard
Primary language French
commented
Status: Active » Postponed

Yes, I do know it can be rather seamless, but this will be a 4.0 version feature, not a 3.0 which is from now just a bugfix branch. Thanks for suggesting.

ducktape’s picture

ducktape commented
StatusFileSize
new redis_cluster_variables.patch3.12 KB

Needed to configure some variables for clustering. Here is a patch that supports some, for phpredis only.

daften’s picture

daften commented
Status: Postponed » Needs review
landsman’s picture

landsman commented

Is this works for D8?

pounard’s picture

pounard
Primary language French
commented

@landsman you should create a new 8.x issue directly, or find for one, @berdir which maintain the 8.x version will probably be able to answer this question.

Rafal Lukawiecki’s picture

Rafal Lukawiecki
Primary language English
Location Ireland
commented

Has any more D7 work been done on supporting connections to cluster-mode of Redis using phpredis?

pounard’s picture

pounard
Primary language French
commented

D7 version is more or less abandoned, I'm sorry I can't give enough time to support it, and we don't use it on production anymore (we have 2 projects left using it, one being rewritten and the other one abandoned). You should ask @berdir which is now the only remaining active maintainer of this module for help, or to name new maintainers for the D7 version.

Rafal Lukawiecki’s picture

Rafal Lukawiecki
Primary language English
Location Ireland
commented

Thank you for your work so far. It looks like this version is still in wide use: 8000 sites use D7 Redis while 18000 use D8. I can see there are a good few patches in the D7 queue, too. Might be an idea to ask someone else to co-maintain to get them committed. I am 100% sure we will be staying with D7 for a good while on our sites.

I am planning to experiment with a simple patch by naively changing the client from phpredis Redis to RedisCluster to see if things could work without having to change anything else. If it works, I will post here.

Either way, many thanks for your support.

pounard’s picture

pounard
Primary language French
commented

I have been very happy to create and maintain this module during those years, but since I don't work with Drupal anymore, it's no my place anymore to maintain it. I'll have a word with @berdir to ask him to search for a D7 maintainer.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

I'd say if someone is interested then they should step up, I don't plan to do any active searching, but feel free to grant someone who is interested the respective permissions.

sujatadrupal’s picture

sujatadrupal commented

Hi All,

I have to connect my Drupal 7 project with Redis Cluster setup.
seeking for help from this community, is there any patch available for this ?
Thanks in advance!

gerson.analista’s picture

gerson.analista commented
StatusFileSize
new support-cluster-connections-2556097-13.patch21.6 KB

I created a patch with support for the PhpRedis RedisCluster class:
https://github.com/phpredis/phpredis/blob/develop/cluster.md

I used the issue patch as a study base:
https://www.drupal.org/project/redis/issues/2900947

I'm still running some tests, but it seems to be working so far.

gerson.analista’s picture

gerson.analista commented
Related issues: +#2900947: Implement initial RedisCluster client integration