o-nikolas commented on code in PR #30380:
URL: https://github.com/apache/airflow/pull/30380#discussion_r1153809658
##########
airflow/cli/commands/task_command.py:
##########
@@ -450,13 +448,12 @@ def task_failed_deps(args):
dep_context = DepContext(deps=SCHEDULER_QUEUED_DEPS)
failed_deps = list(ti.get_failed_dep_statuses(dep_context=dep_context))
- # TODO, Do we want to print or log this
if failed_deps:
- print("Task instance dependencies not met:")
Review Comment:
I had a look at some of the other `*_command.py` modules and many of them
use `print(...)`. I wonder if there is some reason for this? Perhaps someone
else from the community has the historical knowledge about that.
Cc. @eladkal @potiuk
--
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]
