amoghrajesh commented on code in PR #53600:
URL: https://github.com/apache/airflow/pull/53600#discussion_r2250646935
##########
providers/standard/src/airflow/providers/standard/operators/python.py:
##########
@@ -75,11 +74,7 @@
from airflow.sdk.execution_time.callback_runner import
ExecutionCallableRunner
from airflow.sdk.execution_time.context import OutletEventAccessorsProtocol
-
- try:
- from airflow.sdk.definitions.context import Context
- except ImportError: # TODO: Remove once provider drops support for
Airflow 2
- from airflow.utils.context import Context
+ from airflow.utils.context import Context
Review Comment:
This import gets reexported to
`task-sdk/src/airflow/sdk/definitions/context.py`'s Context.
Maybe just use that directly?
##########
airflow-core/src/airflow/models/taskinstance.py:
##########
@@ -367,7 +367,7 @@ def _get_email_subject_content(
else:
from airflow.sdk.definitions._internal.templater import
SandboxedEnvironment
- from airflow.utils.context import context_merge
+ from airflow.utils.context import Context
Review Comment:
This is redirecting to sdk's context. Maybe in a follow up, we should
discuss and separate it 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]
