CaptainAni187 commented on issue #72319: URL: https://github.com/apache/airflow/issues/72319#issuecomment-5586088480
Reproduced on main and opened #72710 with the widening you suggested. I went looking for what actually emits the bare int and couldn't pin it down from inside the repo — `serialize()` writes `total_seconds()` as a float, `BaseSerialization` does the same, and a JSON round-trip keeps `2700.0` intact. So I've written the PR as a defensive widening rather than claiming to have traced your traceback. If you happen to know which producer the payload came through, that'd be useful to note on the PR. Either way the value is clearly reachable, and the `datetime` branch a few lines above already accepts `int | float` for the same kind of cross-producer reason, so accepting it one type down seems consistent. Test is parametrised over int/float/str — on main only the int case fails, which keeps it pinned to the actual broken path. -- 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]
