apilaskowski commented on code in PR #30485: URL: https://github.com/apache/airflow/pull/30485#discussion_r1159475674
########## airflow/cli/commands/celery_command.py: ########## @@ -18,13 +18,16 @@ """Celery command.""" from __future__ import annotations +import logging from contextlib import contextmanager from multiprocessing import Process +from sys import stderr as system_stderr, stdout as system_stdout Review Comment: I intended it to be easier distinguishable from `args.stderr` etc. Inside the `worker` function there is a local variable, which is called `stderr` and when I was working on that I wanted to have them easy to differentiate. If it is ok, I will use version with `sys.stderr` instead. -- 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]
