mobuchowski commented on PR #40854:
URL: https://github.com/apache/airflow/pull/40854#issuecomment-2236502492

   Why not attach it to some new facet as `AirflowDagRunFacet`? Basically, 
`AirflowRunFacet` is tightly coupled to what we have during task instance 
execution, and don't think making it just a bag of possible things makes sense.
   
   I'd rather have the same DagRun information attached to different facets in 
TaskInstance level events and DagRun events.


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