-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow configuration of RedisBackend's health_check_interval #6666
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6666 +/- ##
==========================================
- Coverage 75.60% 70.46% -5.14%
==========================================
Files 138 138
Lines 16377 16497 +120
Branches 2052 2074 +22
==========================================
- Hits 12382 11625 -757
- Misses 3777 4669 +892
+ Partials 218 203 -15
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
This pull request introduces 1 alert when merging 6c7148b into f091bab - view on LGTM.com new alerts:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
thanks, both of you! |
Description
馃憢 Hey folks,
Was working through a reliability issue on our end with our Redis connections and came across the need to update the
health_check_interval
for the underlying Redis connection. The goal here is a simple implementation that allows reading this value via environment variable (redis_backend_health_check_interval
) and passing this through as akwarg
to the underlyingredis-py
module.This is my proposed solution to #6665. Please let me know what you think and thank you for the great library.
Cheers,
Chris