uranusjr commented on code in PR #38180:
URL: https://github.com/apache/airflow/pull/38180#discussion_r1540569553
##########
airflow/models/taskinstance.py:
##########
@@ -2306,8 +2306,10 @@ def emit_state_change_metric(self, new_state:
TaskInstanceState) -> None:
elif new_state == TaskInstanceState.QUEUED:
metric_name = "scheduled_duration"
if self.start_date is None:
- # same comment as above
- self.log.warning(
+ # changing the logging level to debug because this warning is
not working correctly
+ # until fields like a `scheduled_dttm` are implemented
Review Comment:
```suggestion
# This check does not work correctly before fields like
`scheduled_dttm` are implemented.
# TODO: Change the level to WARNING once it's viable.
```
--
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]
