[GitHub] [airflow] kaxil commented on a diff in pull request #25528: Add the option of sending the DAG parser logs to stdout

[GitBox <gi...@apache.org>]

kaxil commented on code in PR #25528:
URL: https://github.com/apache/airflow/pull/25528#discussion_r938740510


##########
airflow/config_templates/airflow_local_settings.py:
##########
@@ -48,6 +49,8 @@
 
 COLORED_FORMATTER_CLASS: str = conf.get_mandatory_value('logging', 'COLORED_FORMATTER_CLASS')
 
+DAG_PROCESSOR_LOG_TARGET: bool = conf.get_mandatory_value('scheduler', 'DAG_PROCESSOR_LOG_TARGET')

Review Comment:
   We tried to consolidate all the logging configs into the logging section a couple of years back: https://github.com/apache/airflow/commit/046b36fab7d1841be63d48758eb2a6a197efaf1c
   
   So that's the goal at least, other configs are marked for deprecated
   
   https://github.com/apache/airflow/blob/d82436b382c41643a7385af8a58c50c106b0d01a/airflow/configuration.py#L173-L196



##########
airflow/config_templates/airflow_local_settings.py:
##########
@@ -48,6 +49,8 @@
 
 COLORED_FORMATTER_CLASS: str = conf.get_mandatory_value('logging', 'COLORED_FORMATTER_CLASS')
 
+DAG_PROCESSOR_LOG_TARGET: bool = conf.get_mandatory_value('scheduler', 'DAG_PROCESSOR_LOG_TARGET')

Review Comment:
   We tried to consolidate all the logging configs into the logging section a couple of years back: https://github.com/apache/airflow/commit/046b36fab7d1841be63d48758eb2a6a197efaf1c
   
   So that's the goal at least, other configs are marked for deprecation
   
   https://github.com/apache/airflow/blob/d82436b382c41643a7385af8a58c50c106b0d01a/airflow/configuration.py#L173-L196



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org