romsharon98 commented on PR #35929: URL: https://github.com/apache/airflow/pull/35929#issuecomment-1841282515
> I think this PR is quite wrong. > > Dag File process does what it does for a good reason. Each of the Dag Files is processed by a separately forked process. If we try to print logs from mulitple DAG file processor processes to the same stdout we will have a total garbage as outputs from those multiple processes will be effectively multiplexed and indistinguishable. > > I tthink @romsharon98 you need to explain what you want to achieve and why you think deleting the code that is responsible for doing it would be a good idea. Thanks for your reply. Last start from the end, I want the dag processor manager logs to print to stdout and to file. Today the logs are print to stdout or to file but not to both of them. To be honest I saw this if statement that making the decision of the output of the logs, tried to remove it and it works. Maybe I miss something or didn't understand deeply the meaning of removing it. If you can guid me what should I do to make it happen in a better way I will appreciate this :) -- 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]
