pankajkoti commented on code in PR #32646:
URL: https://github.com/apache/airflow/pull/32646#discussion_r1278316066
##########
airflow/config_templates/airflow_local_settings.py:
##########
@@ -171,6 +171,9 @@
},
}
+# Task log shipping from Airflow components like scheduler, executor, etc.
+TASK_LOG_SHIPPER_ENABLED = conf.getboolean("logging",
"enable_task_log_shipper")
Review Comment:
yes, after rethinking and discussing with @dstandish , it feels like all
these verbs indicate they are shipping/dispatching/exporting task logs to some
service, but in fact, what we're doing here is sending logs to the task's
context.
Hence, refactored the class from TaskLogShipper to TaskContextLogger, so it
means it logs to the task's context. Hope this renaming is received better.
--
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]
