[GitHub] Fokko commented on issue #4207: [WIP] Run celery integration test with redis broker.

[GitBox <gi...@apache.org>]

Fokko commented on issue #4207: [WIP] Run celery integration test with redis broker.
URL: https://github.com/apache/incubator-airflow/pull/4207#issuecomment-439705915
 
 
   Hi @jmcarp 
   
   Thanks for picking this up. Preferably you run the tests against your own Travis instance first, instead of using the Apache Travis. The hours of Apache Travis are paid, and on your own fork is for free :-)
   
   I don't see any changes in the tests itself. Maybe it would be nice to add a test like: Run a dag with a RedisSensor, and make sure that it will pass. So we have some actual code which communicates with Travis (and therefore also uses the RedisHook).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services