[GitHub] [airflow] potiuk commented on pull request #17576: Add pre/post execution hooks

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

potiuk commented on pull request #17576:
URL: https://github.com/apache/airflow/pull/17576#issuecomment-898028425


   > Is the goal of this PR to replace & generalize this use case or the real issue you are trying to solve is something else ?
   
   Something completely different. Pre/postoperators in Vertica are SQL statements called before/after (Those are strings rather than callables)
   
   What this PR tries to achieve is to provide an option to override pre_/post_execute methods from the base operator with ones provided as constructor parameters. While it was already possible to  override them by extending existing operators, it makes it a bit easier (and with less mental barriers for DAG writers) to provide such methods when you create task in DAG. While previously it required to create a new class and use the new class as operator, this on allows to create methods and pass them as parameters of the cosnstructor - which is easier for many users who write the DAGs.
   


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

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org