ferruzzi commented on PR #70370: URL: https://github.com/apache/airflow/pull/70370#issuecomment-5447801386
Your reasoning on 23 August was right at the time. #71802 didn't have any reviews, so not depending on it was the sensible call. #71802 should be merged shortly, which makes rebasing onto it the right call now. With `SerializedVariableInterval` in core, `resolve_deadline_alert_interval()` no longer imports anything from the SDK, so Amogh's coupling concern is gone once that merges and this gets rebased. My 25 August comment about `resolve()` raising `ValueError` and aborting the whole clear still stands, though. -- 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]
