Skip to content

Source odbc provider to use airflow.sdk.configuration.conf #59995

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
potiuk merged 3 commits into from
Jan 4, 2026
+19 −15
Merged

Source odbc provider to use airflow.sdk.configuration.conf #59995

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c9606ac
Migrate odbc provider to use airflow.sdk.configuration.conf
sunank200 Dec 31, 2025
35e4515
Add common.compat optional dependency to odbc provider
sunank200 Dec 31, 2025
21c3361
Move apache-airflow-providers-common-compat from optional to required…
sunank200 Jan 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 16 additions & 14 deletions providers/odbc/docs/index.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -97,14 +97,15 @@ Requirements

The minimum Apache Airflow version supported by this provider distribution is ``2.11.0``.

======================================= =====================================
PIP package Version required
======================================= =====================================
``apache-airflow`` ``>=2.11.0``
``apache-airflow-providers-common-sql`` ``>=1.20.0``
``pyodbc`` ``>=5.0.0; python_version < "3.13"``
``pyodbc`` ``>=5.2.0; python_version >= "3.13"``
======================================= =====================================
========================================== =====================================
PIP package Version required
========================================== =====================================
``apache-airflow`` ``>=2.11.0``
``apache-airflow-providers-common-compat`` ``>=1.10.1``
``apache-airflow-providers-common-sql`` ``>=1.20.0``
``pyodbc`` ``>=5.0.0; python_version < "3.13"``
``pyodbc`` ``>=5.2.0; python_version >= "3.13"``
========================================== =====================================

Cross provider package dependencies
-----------------------------------
Expand All @@ -116,14 +117,15 @@ You can install such cross-provider dependencies when installing from PyPI. For

.. code-block:: bash

pip install apache-airflow-providers-odbc[common.sql]
pip install apache-airflow-providers-odbc[common.compat]


============================================================================================================ ==============
Dependent package Extra
============================================================================================================ ==============
`apache-airflow-providers-common-sql <https://airflow.apache.org/docs/apache-airflow-providers-common-sql>`_ ``common.sql``
============================================================================================================ ==============
================================================================================================================== =================
Dependent package Extra
================================================================================================================== =================
`apache-airflow-providers-common-compat <https://airflow.apache.org/docs/apache-airflow-providers-common-compat>`_ ``common.compat``
`apache-airflow-providers-common-sql <https://airflow.apache.org/docs/apache-airflow-providers-common-sql>`_ ``common.sql``
================================================================================================================== =================

Downloading official packages
-----------------------------
Expand Down
2 changes: 2 additions & 0 deletions providers/odbc/pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,7 @@ requires-python = ">=3.10"
# After you modify the dependencies, and rebuild your Breeze CI image with ``breeze ci-image build``
dependencies = [
"apache-airflow>=2.11.0",
"apache-airflow-providers-common-compat>=1.10.1", # use next version
"apache-airflow-providers-common-sql>=1.20.0",
"pyodbc>=5.0.0; python_version < '3.13'",
"pyodbc>=5.2.0; python_version >= '3.13'",
Expand All @@ -69,6 +70,7 @@ dev = [
"apache-airflow",
"apache-airflow-task-sdk",
"apache-airflow-devel-common",
"apache-airflow-providers-common-compat",
"apache-airflow-providers-common-sql",
# Additional devel dependencies (do not remove this line and add extra development dependencies)
]
Expand Down
2 changes: 1 addition & 1 deletion providers/odbc/src/airflow/providers/odbc/hooks/odbc.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ def sqlalchemy_scheme(self) -> str:
def driver(self) -> str | None:
"""Driver from init param if given; else try to find one in connection extra."""
extra_driver = self.connection_extra_lower.get("driver")
from airflow.configuration import conf
from airflow.providers.common.compat.sdk import conf

if extra_driver and conf.getboolean("providers.odbc", "allow_driver_in_extra", fallback=False):
self._driver = extra_driver
Expand Down