dstandish commented on code in PR #43936:
URL: https://github.com/apache/airflow/pull/43936#discussion_r1847011478
##########
providers/src/airflow/providers/openlineage/plugins/listener.py:
##########
@@ -325,15 +338,22 @@ def on_failure():
logical_date=dagrun.logical_date,
)
+ if not task_instance.queued_dttm:
+ # This should not happen outside of tests, but if it does, we
should log it and continue
Review Comment:
why should we log it? does it really matter? i would not bother if it's
truly unexpected. wouldn't it cause problems and be surfaced elsewhere? in
that case, this is just noise.
--
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]
