shahar1 commented on PR #69853: URL: https://github.com/apache/airflow/pull/69853#issuecomment-6239792916
Thanks for the contribution, @qotique, and sorry for the long wait on this one. I'm closing this PR because the linked issue #68908 was closed as not planned. The agreed approach there is to invoke the function and then wait with the deferrable `HttpSensor`, which already works without new provider code. There are also two design problems that review fixes can't solve. First, `functions.call` is a synchronous, rate-limited API meant only for testing. Running it in `asyncio.to_thread` just moves the blocking call from the worker to the triggerer. Second, there is no execution handle to resume from, so a triggerer restart would call the function again. If you'd like to keep contributing in this area, a system-test example of the invoke + deferrable `HttpSensor` pattern (see the issue discussion) would be very welcome. --- Drafted-by: Claude Code (Opus 5.5); reviewed by @shahar1 before posting -- 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]
