[GitHub] [airflow] Taragolis commented on pull request #23872: DagFileProcessorManager: Start a new process group only if current process not a session leader

["Taragolis (via GitHub)" <gi...@apache.org>]

Taragolis commented on PR #23872:
URL: https://github.com/apache/airflow/pull/23872#issuecomment-1457097938

   @toxadx Feel free to open a PR (do not need to create issue, Airflow project is PR-friendly), for more details see [Contributor's Quick Start ](https://github.com/apache/airflow/blob/main/CONTRIBUTORS_QUICK_START.rst)
   
   > According to "Ask forgiveness not permission" concept I would like to propose to rewrite set_new_process_group:
   
   There is at least 3 different reason could be in Linux/macOS and original PR only cover first one, in the other cases  I guess it should still raise original error, however may be we should ignore all errors 
   
   ```console
   [EPERM] The process indicated by the pid argument is a session leader.
   [EPERM] The effective user ID of the requested process is different from that of the caller and the process is not a descendant of the calling process.
   [EPERM] The value of the pgid argument is valid, but does not match the process ID of the process indicated by the pid argument and there is no process with a process group ID that matches the value of the pgid argument in the same session as the calling process.
   ```
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org