molcay commented on PR #41329: URL: https://github.com/apache/airflow/pull/41329#issuecomment-2304139636
Hi @potiuk, Yes, it was on purpose. I was reading the first discussion and there is this comment written by Taragolis: https://github.com/apache/airflow/issues/35474#issuecomment-1801897704 I was trying to implement the 2nd step also. If you think this is too complicated for single commit, I can split the change into two commits: - 1 commit for the changes in signaling (reap_process_group function) - 1 commit for the changes in `local_task_job_runner.py`. Also, One test case (with 4 different parameter groups) is failing. I could not fix the problem with that test case. If you can give me some help, would be very much appreciated! -- 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]
