seanghaeli commented on PR #66608: URL: https://github.com/apache/airflow/pull/66608#issuecomment-4410197486
## Verification (corrected) Ran in Breeze (Postgres, full migrations) to verify the comms plumbing works e2e: - Confirmed `GetDagRun` round-trips through the triggerer's `ToTriggerSupervisor` → `_handle_request` → `DagRunResult` response path without breaking existing trigger handling - Verified `SUPERVISOR_COMMS.asend()` is the correct async calling pattern — uses `TriggerCommsDecoder` from `init_comms()` with async lock for coroutine safety in the trigger event loop - Verified the `DagRun` generated model has all fields accessed in `_build_context`: `logical_date`, `data_interval_start`, `data_interval_end`, `conf` - Backward compat confirmed: old callbacks with stored `"context"` key (queued before this change) still work --- Drafted-by: Claude Code (Opus 4.6); reviewed by @seanghaeli 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]
