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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to