bolkedebruin commented on issue #17984:
URL: https://github.com/apache/airflow/issues/17984#issuecomment-956272791


   I thin it is okay to streach the defintion of lineage like this. However, 
then I would not call it data lineage anymore. It is more like a generic 
auditing system then as it not only tracks data but starts tracking tasks as 
well. We can expose this kind of information of course. `post_execute` seems 
ambigious in its defintion as it is unclear whether is is ran on success only. 
I prefer airflow to have a `on_state_change(old, new, context))` function which 
we can decorate with a plugin kind of way to handle this so you can tie in the 
LineageBackend to register to these state changes.
   
   Does that make sense?


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


Reply via email to