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 2019/09/26 20:49:09 UTC

[GitHub] [airflow] szczeles commented on issue #5428: AIRFLOW-4809 | s3_delete_objects_operator should not fail on empty list of keys

szczeles commented on issue #5428: AIRFLOW-4809 | s3_delete_objects_operator should not fail on empty list of keys
URL: https://github.com/apache/airflow/pull/5428#issuecomment-535680378
 
 
   After a few experiments I implemented this using [ShortCircuitOperator](https://github.com/apache/airflow/blob/93e856e7021a4b8c60afd737689415ffb1015d7c/airflow/operators/python_operator.py#L169) by skipping delete operator if list of files is empty. It's more straightforward solution and doesn't require adding any extra params in the current operators.
   
   Therefore I'm closing this PR. Thanks for your review!

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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