You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "steveahnahn (via GitHub)" <gi...@apache.org> on 2025/12/01 00:46:52 UTC

Re: [PR] fix string to datetime pydantic and test [airflow]

steveahnahn commented on PR #58791:
URL: https://github.com/apache/airflow/pull/58791#issuecomment-3594086622

   @amoghrajesh You're right that at runtime it returns datetime. This is because when the response travels from supervisor to task runner, it gets deserialized where Pydantic validation converts the string to datetime. As for client.py, it was returning a string with model_construct() which bypasses validation, and the test itself was testing against a string and passing. This fix ensures the client returns the correct type rather than relying on downstream


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org