You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2018/12/13 08:17:54 UTC

[GitHub] Fokko commented on issue #4068: [AIRFLOW-2310]: Add AWS Glue Job Compatibility to Airflow

Fokko commented on issue #4068: [AIRFLOW-2310]: Add AWS Glue Job Compatibility to Airflow
URL: https://github.com/apache/incubator-airflow/pull/4068#issuecomment-446880175
 
 
   Like stated earlier. My preference would be to have the operator itself poll for it. If you think about atomicity, if the job fails, you will need to clear both the sensor and the operator. This makes life more complicated and adds very little value. Also, your dags will be twice as big.
   
   Please fix the flake8 issues: `./tests/contrib/operators/test_aws_glue_job_operator.py:42:9: F841 local variable 'some_script' is assigned to but never used`

----------------------------------------------------------------
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