fat-catTW commented on code in PR #74074: URL: https://github.com/apache/airflow/pull/74074#discussion_r4184805951
########## task-sdk/src/airflow/sdk/execution_time/supervisor.py: ########## Review Comment: Thanks for contribution! A Minor docs consistency: this docstring still says `models.Variable.get` checks for the presence of `task_runner.SUPERVISOR_COMMS` to decide whether it is in a Task SDK execution context. With this PR, Variable/Connection now go through `in_task_execution_context()` / `task_runner.supervisor_comms()` instead. Could we refresh this wording so future readers are not pointed back to the old direct-presence check? -- 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]
