[GitHub] jmcarp commented on issue #4207: [AIRFLOW-3367] Run celery integration test with redis broker.

jmcarp commented on issue #4207: [AIRFLOW-3367] Run celery integration test with redis broker. URL: https://github.com/apache/incubator-airflow/pull/4207#issuecomment-443298286 Ping @ashb. Hopefully this will be quick to review, since most of the diff is indenting for context managers.

[GitHub] jmcarp commented on issue #4207: [AIRFLOW-3367] Run celery integration test with redis broker.

jmcarp commented on issue #4207: [AIRFLOW-3367] Run celery integration test with redis broker. URL: https://github.com/apache/incubator-airflow/pull/4207#issuecomment-442699347 Conflicts fixed, ready for another look @ashb.

[GitHub] jmcarp commented on issue #4207: [AIRFLOW-3367] Run celery integration test with redis broker.

jmcarp commented on issue #4207: [AIRFLOW-3367] Run celery integration test with redis broker. URL: https://github.com/apache/incubator-airflow/pull/4207#issuecomment-441782392 Thanks for reviewing @ashb! I updated the tests like you suggested.

[GitHub] jmcarp commented on issue #4207: [AIRFLOW-3367] Run celery integration test with redis broker.

jmcarp commented on issue #4207: [AIRFLOW-3367] Run celery integration test with redis broker. URL: https://github.com/apache/incubator-airflow/pull/4207#issuecomment-441499749 Rebased and fixed the conflict with #4090. No rush, but this should be ready for review when you have time