abhid-007 opened a new pull request, #66990: URL: https://github.com/apache/airflow/pull/66990
`BigQueryIntervalCheckTrigger.__init__` accepts `dataset_id`, `table_id`, and `poll_interval` and stores them via the parent `BigQueryInsertJobTrigger.__init__`, but `serialize()` omitted all three. After a triggerer restart the trigger was reinstantiated with defaults (`dataset_id=None`, `table_id=None`, `poll_interval=4.0`), silently changing polling cadence and dropping the templated table identity for any task that customised them. This PR adds the three fields to the `serialize()` payload so the trigger round-trips through `serialize()` -> `BigQueryIntervalCheckTrigger(**kwargs)` cleanly. The existing serialization test gains the three new keys, and a round-trip test mirrors the pattern used by the sibling fixes in #66965 / #66966 / #66968. This is the last of the five triggers called out in #66961; the static check that flags this class of bug is being introduced in #66960. related: #66961 related: #66960 --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Opus 4.7) Generated-by: Claude Code (Opus 4.7) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- 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]
