1fanwang commented on PR #66914: URL: https://github.com/apache/airflow/pull/66914#issuecomment-4685697082
You're right, and the race condition is the deciding factor for me: the explicit recompute hook only fires on the unpause paths I wired it into. Anything that flips `is_paused` out-of-band — DB, a plugin, an auth manager, some future code path — leaves the scheduler reading stale dates until the next parse cycle. I can't make that invariant airtight across every backend surface, so freezing the value at the source isn't the right shape. Going to step back and rethink rather than iterate on this. The ORM-level proxy you floated (real values when unpaused, `None` when paused) keeps the invariant at a single read chokepoint instead of scattering it across every writer — a much smaller surface, and worth exploring on its own. Thanks for the review, Pierre. -- 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]
