potiuk commented on pull request #19361: URL: https://github.com/apache/airflow/pull/19361#issuecomment-981937009
> Along these lines, one thing that might be more palatable is to add to BaseOperator the ability to inject (using params) a callable that would be called in `pre_execute`. E.g. call it `pre_execute_hook: Optional[Callable]=None`. Then users could call arbitrary logic in pre_execute without having to subclass. And this would open up some interesting possibilities where we could inject pre_execute and post_execute callables with global task policy, which I think could be very useful. Well, @dstandish. We've added it in https://github.com/apache/airflow/pull/17576 (after discussion with @malthe) and then there was a follow-up discussion as results of which @ashb marked it as experimental (see Ash's comment there https://github.com/apache/airflow/pull/18140) I wonder what's your opinion :P -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
