-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
[AIRFLOW-3353] Upgrade redis client #4834
Conversation
Now that Celery/Kombu have updated and work with RedisPy 3.x (they in fact force us to use 3.2) we should re-introduce this change.
Note to whoever merges this - please remove the |
lgtm, I am afk, and I think @ashb you could merge it on your own if you are by your computer. |
Thanks @ashb for picking this up so quickly |
Now that Celery/Kombu have updated and work with RedisPy 3.x (they in fact force us to use 3.2) we should re-introduce this change.
@ashb , I assume once other committers give +1, it should be fine. Besides we currently have a limited set of active committers. |
Now that Celery/Kombu have updated and work with RedisPy 3.x (they in fact force us to use 3.2) we should re-introduce this change.
Now that Celery/Kombu have updated and work with RedisPy 3.x (they in fact force us to use 3.2) we should re-introduce this change.
Make sure you have checked all steps below.
Jira
https://issues.apache.org/jira/browse/AIRFLOW-3353
Description
Now that Celery/Kombu have updated and work with RedisPy 3.x (they in
fact force us to use 3.2) we should re-introduce this change.
This re-introduces the change from #4203 (with the addition of a note in UPDATING.md. The tests have been changed since so don't need updating anymore) that was reverted in #4205
Tests
Commits
Documentation
Code Quality
flake8