Shoryamishra61 commented on issue #74544:
URL: https://github.com/apache/airflow/issues/74544#issuecomment-6124271981

   Thanks for catching this. Agreed that sharing the lazily initialized adapter 
across multiple API emission threads introduces a concurrency risk.
   
   I'll constrain the API-server executor to a single worker while preserving 
the scheduler's existing ProcessPoolExecutor configuration, and add regression 
coverage for configurations where `dag_state_change_process_pool_size > 1`, 
concurrent submissions, and worker shutdown.
   
   Appreciate the review --  I'd prefer to keep the fix focused on the original 
process-leak issue without introducing a separate adapter ownership problem.


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

Reply via email to