Skip to content

allow reading from a disconnected read-only slave#1460

Open
ttlanhil wants to merge 1 commit into
redis:masterfrom
ttlanhil:master
Open

allow reading from a disconnected read-only slave#1460
ttlanhil wants to merge 1 commit into
redis:masterfrom
ttlanhil:master

Conversation

@ttlanhil

Copy link
Copy Markdown

Pull Request check-list

  • Does npm test pass with this change (including linting)?
    The list of failing tests is unchanged / pull request does not cause new failures
  • Is the new or changed code fully tested?
    Tested, but not covered by test files
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Description of change

My company has local read-only redis slaves that are allowed to cache stale data; so we needed a new option to "allow_disconnected_slaves":
If set to true and the redis server is configured as slave-read-only then connections to a disconnected slave are allowed. Otherwise connections will be retried until the master connection is restored.

ttlanhil pushed a commit to ttlanhil/DefinitelyTyped that referenced this pull request Sep 18, 2019
ttlanhil pushed a commit to ttlanhil/DefinitelyTyped that referenced this pull request Sep 18, 2019
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.

1 participant