jason810496 commented on code in PR #53122:
URL: https://github.com/apache/airflow/pull/53122#discussion_r2210008234
##########
airflow-core/src/airflow/executors/local_executor.py:
##########
@@ -60,6 +59,12 @@ def _run_worker(
log = logging.getLogger(logger_name)
log.info("Worker starting up pid=%d", os.getpid())
+ if sys.platform == "darwin":
+ log.debug("Mac OS detected, skipping setproctitle")
+ setproctitle = lambda title: None
+ else:
+ from setproctitle import setproctitle
Review Comment:
> We can still leave that at the top of each file to avoid duplication
within the same file. (which shouldn't bring any cross component dependencies)
Sure! Agree with top level approach to void duplication in same module
level.
--
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]
