You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "ashb (via GitHub)" <gi...@apache.org> on 2025/03/21 14:56:00 UTC

Re: [PR] Use time.time() for dag file process start time [airflow]

ashb commented on PR #48004:
URL: https://github.com/apache/airflow/pull/48004#issuecomment-2743613684

   https://github.com/giampaolo/psutil/blob/a17550784b0d3175da01cdb02cee1bc6b61637dc/psutil/__init__.py#L766-L773
   
   If you are worried about process reuse and the start_time being wrong, then call `proc.create_time()` once first.
   
   However the chance of process reuse happening and being the cause of the bugs is tiny. I'm not saying it can't happen, but that the for it to, a large number of processes need to have been created, enough for the pid to overflow and wrap around. I don't know what the limit is, but I suspect it's somewhere around 64k processes on OSX.


-- 
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