Does the client work for Elasticache cluster mode enabled #20

Activity

1. funny-falcon commented on Mar 8, 2022

   

   funny-falcon

   on Mar 8, 2022

   Contributor

   More actions

   Why do you think it doesn't maintain slot to node mapping?
   rediscluster folder is explicitly for that.
2. abinavv20 commented on Mar 9, 2022

   

   abinavv20

   on Mar 9, 2022

   Last edited by abinavv20

   Author

   More actions

   Our redis metrics indicate that only the primary is accepting writes and replica is accepting reads. The primary usually accepts writes and reads. Does it have to do with this line of code here? https://github.com/joomcode/redispipe/blob/master/rediscluster%2Fcluster.go#L436-L441

   
3. funny-falcon commented on Mar 9, 2022

   

   funny-falcon

   on Mar 9, 2022

   Contributor

   More actions

   You can set policy with Cluster.WithPolicy:
   

   redispipe/rediscluster/rw_policy.go

   Line 26 in 1283dc7

   func (c *Cluster) WithPolicy(policy ReplicaPolicyEnum) PolicyMan {

   I don't get which way read commands go to replica if you didn't specify policy, since you correctly mention MasterOnly is primary policy.
4. abinavv20 commented on Mar 10, 2022

   

   abinavv20

   on Mar 10, 2022

   Last edited by abinavv20

   Author

   More actions

   I will try this out.

   I believe the default behavior should be primary accepting writes and reads and replica is accepting only reads, even without specifying policy. This is the behavior of redis cluster mode.

   Can I bring this up as an issue for bug fix?