uranusjr commented on code in PR #59922: URL: https://github.com/apache/airflow/pull/59922#discussion_r2660505767
########## airflow-core/newsfragments/59835.significant.rst: ########## @@ -1,5 +1,6 @@ Methods removed from TaskInstance -On class ``TaskInstance``, functions ``run()``, ``render_templates()``, and -private members related to them have been removed. The class has been -considered internal since 3.0, and should not be relied on in user code. +On class ``TaskInstance``, functions ``run()``, ``render_templates()``, +``get_template_context()``, and private members related to them have been +removed. The class has been considered internal since 3.0, and should not be +relied on in user code. Review Comment: Ah cluster policies, that makes sense. You shouldn’t need to whole context for that, but I can see you’d want some of it for some cases. What exactly the replacement should be depends on the use case, and with execution-time now being strictly separated from he scheduler, the context should be different objects now (so it does not make sense to be able to generate the runtime context there). Give me a ping if this comes up and we’ll figure this out. -- 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]
