Skip to content

Move JDBC Provider to the New Structure#46269

Merged
potiuk merged 1 commit intoapache:mainfrom
bugraoz93:feat/46045/move-jdbc
Feb 1, 2025
Merged

Move JDBC Provider to the New Structure#46269
potiuk merged 1 commit intoapache:mainfrom
bugraoz93:feat/46045/move-jdbc

Conversation

@bugraoz93
Copy link
Copy Markdown
Contributor

related: #46045

No PR is created at this moment.


^ 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 newsfragments.

@potiuk
Copy link
Copy Markdown
Member

potiuk commented Jan 30, 2025

Conflicts again - but this time I think 🤞 all of those flaky tests shoudl be handled.

@bugraoz93 bugraoz93 force-pushed the feat/46045/move-jdbc branch from 1970921 to 10c4702 Compare February 1, 2025 13:31
@bugraoz93 bugraoz93 force-pushed the feat/46045/move-jdbc branch from 10c4702 to d4eec36 Compare February 1, 2025 13:32
@bugraoz93
Copy link
Copy Markdown
Contributor Author

Conflicts again - but this time I think 🤞 all of those flaky tests shoudl be handled.

Thanks for checking and syncing! 🤞

Copy link
Copy Markdown
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI Error seems to be in Google Operator (from main) and not from this change. I think we can merge.

@bugraoz93
Copy link
Copy Markdown
Contributor Author

CI Error seems to be in Google Operator (from main) and not from this change. I think we can merge.

Yes, I am going to check if I can fix that in another PR. First will check if anyone is already working on a fix. Thanks for your reviews!

@potiuk
Copy link
Copy Markdown
Member

potiuk commented Feb 1, 2025

Indeed. Can't fix those quickly but if somoene could (looks like just indenting of docs in bigquery.rst) it would be great.

@potiuk potiuk merged commit edd60c7 into apache:main Feb 1, 2025
@bugraoz93
Copy link
Copy Markdown
Contributor Author

It seems like no one is working. I will make the changes.

amoghrajesh pushed a commit to astronomer/airflow that referenced this pull request Feb 3, 2025
dabla pushed a commit to dabla/airflow that referenced this pull request Feb 3, 2025
niklasr22 pushed a commit to niklasr22/airflow that referenced this pull request Feb 8, 2025
ambika-garg pushed a commit to ambika-garg/airflow that referenced this pull request Feb 17, 2025
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