kosiew commented on PR #64960: URL: https://github.com/apache/airflow/pull/64960#issuecomment-4573109369
@moomindani I [switched the helper back to `airflow.sdk.serde.serialize`, kept the Databricks-specific `ValueError` wrapping, and added coverage for a serde-supported datetime value so Tenacity retry strategy objects still fail fast while Airflow-serde-compatible values pass](https://github.com/apache/airflow/pull/64960/commits/557669b4543419918db7a07771e60134890b60b1). On the minor points: the local retry test helper was removed to follow the [earlier review preference](https://github.com/apache/airflow/pull/64960#discussion_r3303987065) to avoid a separate shared helper and keep the retry test data local; the fixed timestamp came from [prior review](https://github.com/apache/airflow/pull/64960#discussion_r3066475885) and is incidental. -- 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]
