Skip to content

Make smembers isMasterOnly = false#232

Open
fehmicansaglam wants to merge 1 commit into
etaty:masterfrom
fehmicansaglam:master
Open

Make smembers isMasterOnly = false#232
fehmicansaglam wants to merge 1 commit into
etaty:masterfrom
fehmicansaglam:master

Conversation

@fehmicansaglam

Copy link
Copy Markdown
Contributor

Fixes #231
SMEMBERS is a read only command therefore it should be run on slaves.

SMEMBERS is a read only command therefore it should be run on slaves.
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.06%) to 90.508% when pulling fcfc71d on fehmicansaglam:master into 5600b00 on etaty:master.

@farico

farico commented Dec 31, 2019

Copy link
Copy Markdown

any updates on this PR and issue @etaty ?

@farico

farico commented Dec 31, 2019

Copy link
Copy Markdown

Temporary solution would be to use slavesClients directly on RedisClientMasterSlaves:

val redis = RedisClientMasterSlaves(...)
redis.slavesClients.smembers() // send SMEMBERS to replicas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Smembers is set as isMasterOnly = true

3 participants