Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Alerts for Redis unavailability #180

Closed
msavin opened this issue Nov 1, 2017 · 3 comments
Closed

Alerts for Redis unavailability #180

msavin opened this issue Nov 1, 2017 · 3 comments

Comments

@msavin
Copy link

msavin commented Nov 1, 2017

It would be cool if this package created some kind of publication or something to notify clients if Redis is unavailable. Ideally, we could run something like Meteor.hasRedis and notify end-users of issues if there are any.

@msavin msavin changed the title Client side alerts for Redis unavailability Alerts for Redis unavailability Nov 1, 2017
@msavin
Copy link
Author

msavin commented Nov 1, 2017

Additionally, maybe create room for a callback:

RedisOplog.onRetry(function (retryCount) {
  if (retryCount > 5) {
    panic();
    callMom();
    disruptCEOMeditationSession();
  }
})

@theodorDiaconu
Copy link
Contributor

@msavin very good idea

@theodorDiaconu
Copy link
Contributor

@msavin this is going to be fixed in the 1.2.3 release I already did it.

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

No branches or pull requests

2 participants