Skip to content

Conversation

@sunank200
Copy link
Collaborator

@sunank200 sunank200 commented Dec 31, 2025

This PR migrates the trino provider to use airflow.sdk.configuration.conf instead of airflow.configuration.conf. This change maintains backward compatibility through the common compat module.

Added conf to Common Compat Module in #59966
related: #60000


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@sunank200 sunank200 force-pushed the migrate-conf-imports-trino branch from d659af1 to 94d6c65 Compare December 31, 2025 14:03
@potiuk potiuk changed the title Source conf from airflow.sdk.configuration.conf for trino providers Source conf from airflow.sdk.configuration.conf for more providers Dec 31, 2025
@sunank200
Copy link
Collaborator Author

This PR is for trino only. It is just that #59966 has to be merged.

@potiuk
Copy link
Member

potiuk commented Dec 31, 2025

ah I see :)

@potiuk potiuk changed the title Source conf from airflow.sdk.configuration.conf for more providers Source conf from airflow.sdk.configuration.conf for trino provider Dec 31, 2025
@sunank200 sunank200 force-pushed the migrate-conf-imports-trino branch from 94d6c65 to c43815d Compare December 31, 2025 15:54
@sunank200 sunank200 marked this pull request as ready for review December 31, 2025 15:54
@sunank200 sunank200 force-pushed the migrate-conf-imports-trino branch 2 times, most recently from b228530 to 6683996 Compare December 31, 2025 16:56
@sunank200 sunank200 force-pushed the migrate-conf-imports-trino branch from 6683996 to 76fb60b Compare December 31, 2025 18:07
@jscheffl jscheffl merged commit c0fc312 into apache:main Dec 31, 2025
99 checks passed
stegololz pushed a commit to stegololz/airflow that referenced this pull request Jan 9, 2026
…pache#59968)

* Migrate trino provider to use airflow.sdk.configuration.conf

* Migrate trino provider to use airflow.sdk.configuration.conf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants