Skip to content

Retry on failure (after reconnecting) #17

Closed as not planned
Closed as not planned
@jeffreyzant

Description

@jeffreyzant

Hi there!

Thanks for creating this (much simpler) version of the sentinel driver. We've started to move away from our pull request (monospice/laravel-redis-sentinel-drivers#36) to this package.

One big difference I've seem to find is that this package does not retry the executed command when the master Redis instance goes away (for instance with a rolling update). Might be interesting to implement it, for instance: https://github.com/monospice/laravel-redis-sentinel-drivers/pull/36/files#diff-3bf85f59585710cebd9ae7787d8b7ffb7df273ce8ffe2e72053ef96b4bb81ccaR256.

I'd be willing to put in the time for the pull request if you are open for it!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions