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 2020/06/01 10:31:32 UTC

[GitHub] [airflow] ashb edited a comment on pull request #8296: Make execute and on_kill as abstractmethod in baseoperator

ashb edited a comment on pull request #8296:
URL: https://github.com/apache/airflow/pull/8296#issuecomment-636771009


   > But if we add on_kill function to all operator, they could kill process if not success
   
   The task runner will handle killing processes already (I think?), so on_kill is only needed for specialized handling, such as canceling an API request.


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