o-nikolas commented on code in PR #36794:
URL: https://github.com/apache/airflow/pull/36794#discussion_r1452725911
##########
airflow/cli/commands/celery_command.py:
##########
@@ -170,7 +170,7 @@ def worker(args):
"--loglevel",
celery_log_level,
"--pidfile",
- worker_pid_file_path,
+ worker_pid_file_path + ".celery.pid",
Review Comment:
Yupp, back when AIP-51 was being worked on initially, the celery executor
loaded its commands from that `celery_command.py` module. That module should
have been moved to the new provider package when that migration was underway
afterwords. But looks like it just got missed in coding and in the PR! The same
is likely true for the kubernetes executor.
--
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]
