This is an automated email from the ASF dual-hosted git repository.

kaxilnaik pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new 87afb93dc09 Provider Migration: Replace `BaseOperator` to Task SDK for 
`samba` (#52588)
87afb93dc09 is described below

commit 87afb93dc09c25be2286371d3be42e328c4d75df
Author: Zhen-Lun (Kevin) Hong <[email protected]>
AuthorDate: Tue Jul 1 03:36:21 2025 +0800

    Provider Migration: Replace `BaseOperator` to Task SDK for `samba` (#52588)
---
 .../providers/samba/transfers/gcs_to_samba.py      |  8 ++---
 .../src/airflow/providers/samba/version_compat.py  | 39 ++++++++++++++++++++++
 2 files changed, 41 insertions(+), 6 deletions(-)

diff --git 
a/providers/samba/src/airflow/providers/samba/transfers/gcs_to_samba.py 
b/providers/samba/src/airflow/providers/samba/transfers/gcs_to_samba.py
index 74120642442..87b83a0c3c8 100644
--- a/providers/samba/src/airflow/providers/samba/transfers/gcs_to_samba.py
+++ b/providers/samba/src/airflow/providers/samba/transfers/gcs_to_samba.py
@@ -25,18 +25,14 @@ from tempfile import NamedTemporaryFile
 from typing import TYPE_CHECKING
 
 from airflow.exceptions import AirflowException
-from airflow.models import BaseOperator
 from airflow.providers.google.cloud.hooks.gcs import GCSHook
 from airflow.providers.samba.hooks.samba import SambaHook
+from airflow.providers.samba.version_compat import BaseOperator
 
 WILDCARD = "*"
 
 if TYPE_CHECKING:
-    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.providers.samba.version_compat import Context
 
 
 class GCSToSambaOperator(BaseOperator):
diff --git a/providers/samba/src/airflow/providers/samba/version_compat.py 
b/providers/samba/src/airflow/providers/samba/version_compat.py
new file mode 100644
index 00000000000..ec6d0191a94
--- /dev/null
+++ b/providers/samba/src/airflow/providers/samba/version_compat.py
@@ -0,0 +1,39 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+from __future__ import annotations
+
+
+def get_base_airflow_version_tuple() -> tuple[int, int, int]:
+    from packaging.version import Version
+
+    from airflow import __version__
+
+    airflow_version = Version(__version__)
+    return airflow_version.major, airflow_version.minor, airflow_version.micro
+
+
+AIRFLOW_V_3_0_PLUS = get_base_airflow_version_tuple() >= (3, 0, 0)
+
+if AIRFLOW_V_3_0_PLUS:
+    from airflow.sdk import BaseOperator
+    from airflow.sdk.definitions.context import Context
+else:
+    from airflow.models import BaseOperator  # type: ignore[no-redef]
+    from airflow.utils.context import Context  # type: ignore[no-redef]
+
+__all__ = ["AIRFLOW_V_3_0_PLUS", "BaseOperator", "Context"]

Reply via email to