henry3260 commented on PR #65587: URL: https://github.com/apache/airflow/pull/65587#issuecomment-4428256780
> How about adding `lifespan` to ensure `SUPERVISOR_COMMS` will be initialized for in-process Execution API? I think lifespan is the wrong scope for this. It initializes app-scoped state once at startup, but the issue here is request-scoped we need to isolate an in-process Execution API request from the outer client context. Initializing `SUPERVISOR_COMMS `in lifespan would make that client marker app-wide and could leak it into server-side request handling wdyt? -- 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]
