vincbeck opened a new pull request, #28476:
URL: https://github.com/apache/airflow/pull/28476

   Migrate `DagFileProcessorManager._deactivate_stale_dags` to Internal API. 
Please while reviewing it to pay double attention to how I handle the logs. 
When discussing with @mhenc, we decided:
   - If a function is NOT executed though internal API, the logger should stay 
as is
   - If a function is executed through internal API, the logger should be 
separate
   
   Very happy to hear your feedback and/or suggestions since, to be very 
honest, I am not a big fan of passing the logger as parameter
   
   Closes #28270 


-- 
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-unsubscr...@airflow.apache.org

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

Reply via email to