kacpermuda commented on PR #40854: URL: https://github.com/apache/airflow/pull/40854#issuecomment-2236869935
> My current plan is: > > * Mark AirflowRunFacet as deprecated, but keep it for backward compatibility (al least for several next releases). > * Add new facet `airflow_task: AirflowTaskRunFacet` which would just copy `AirflowRunFacet`, but with a new name and key. > * Add new facet `airflow_dag: AirflowDagRunFacet` which would implement changes for [OpenLineage: Add AirflowRunFacet to DAG run events #40798](https://github.com/apache/airflow/issues/40798). > > What do you think? I think i would proceed with the last point for now in this PR, and create a discussion for the first two so that we can decide if / how we want to make it less confusing for users. The AirflowRunFacet is the primary facet that users rely on, so we need to handle it with care. Before proceeding with any actions, we should have an in-depth discussion, as deprecating it without such a discussion would not be advisable. 😄 We can also discuss here what would be the best key for this new facet: `airflowDag`, `airflowRun`, `airflowDagRun`? -- 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]
