ferruzzi commented on PR #70370: URL: https://github.com/apache/airflow/pull/70370#issuecomment-5736816845
@hkc-8010 72651merged this morning which makes this one the head of the snake with 5 more blocked behind it. I'd love to get this one rebased, re-reviewed, and merged ASAP. @amoghrajesh - I think your 5 August concern is already retired. You flagged the `isinstance(interval, VariableInterval)` runtime coupling; hkc pointed that to the core `SerializedVariableInterval` back on 10 September, and the only remaining runtime import of the SDK class was on main, which #72651 removed by mapping the classname onto `SerializedVariableInterval` instead. That deletion should get pulled in with the next rebase. I don't think there's any SDK coupling left in this 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]
