AP24110011695 commented on PR #76629: URL: https://github.com/apache/airflow/pull/76629#issuecomment-6712003192
Thanks for tracing every consumer, @Abolfazlrwm — this is exactly the diligence that makes the substitution safe to accept: the datamodel conversion happens inside the before-validator, the eager loads cover the only relationship the conversion touches, and there's no case where the reload changes previously-working behavior. Also good to know the CI failures trace to the pre-#76632 flake rather than this change. LGTM from my side — looks ready once CI confirms on the updated branch. -- 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]
