-
Notifications
You must be signed in to change notification settings - Fork 150
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
Cache error fallback (ters) #643
Comments
@slukes - thanks for reporting this. It looks like there has been little effort around this on handling it. I think that for right now we dont plan to fix this but if you want to come up with a solution we would love to look at it. Are you thinking that this would be an event that is fired when redis is down? Going to close this out and if you want to create an enhancement go ahead and do a pull request. |
Hello @jaredwray what I was imagining would be to:
WDYT? |
@slukes - would we want to just do the |
Let's move this discussion to a PR 😄 |
Describe the bug
Following #588 and #51
It still appears to be the case that if redis is down, we get an error rather than the wrapped function being called.
Could we get some feedback on this please?
Would be happy to invest 1/2 a day to a day to help if you could give any guidance on how you imagined fixing this.
Thanks!
The text was updated successfully, but these errors were encountered: