ferruzzi commented on code in PR #62645:
URL: https://github.com/apache/airflow/pull/62645#discussion_r2992158707
##########
task-sdk/src/airflow/sdk/execution_time/supervisor.py:
##########
@@ -2000,7 +1999,7 @@ def _configure_logging(log_path: str, client: Client) ->
tuple[FilteringBoundLog
return logger, log_file_descriptor
-def supervise(
+def supervise_task(
Review Comment:
I think I disagree on this one. That's how deprecation warnings should
work, right? It signals to the provider that they should update their call
from `supervise()` to `supervise_task()`. @wjddn279 already has a draft PR at
[#63498](https://github.com/apache/airflow/pull/63498) to update Edge to use
the new API, and is waiting for this to be finalized so we can get that
adjusted and merged.Z
We have community members volunteered to update all of the executors with
PRs in draft waiting for this, so it should be a near-immediate adoption once
this gets merged. Full list here:
https://github.com/apache/airflow/issues/62887
--
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]
