dstandish commented on code in PR #43936:
URL: https://github.com/apache/airflow/pull/43936#discussion_r1849249707
##########
providers/src/airflow/providers/dbt/cloud/utils/openlineage.py:
##########
@@ -156,6 +156,8 @@ async def get_artifacts_for_steps(steps, artifacts):
task_id=operator.task_id,
logical_date=_get_logical_date(task_instance),
try_number=_get_try_number(task_instance),
+ queued_dttm=task_instance.queued_dttm,
Review Comment:
I just mean call on_task_instance_running the first time but not in
subsequent times. Or essentially, like, let the duplicate signals get ignored.
What's the point of having say 10 OL start events for the same TI? it's sort
of an implementation detail that we suspend and resume sensors with reschedule
mode. nothing new is happening each time. it's still in effect one long
continuous running phase, just with a suspend-resume sort of sub-phase, you
know?
--
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]
