Skip to content

Conversation

@guan404ming
Copy link
Member

Related Issue

Why

We supported polars in #48875.

How

  • This PR is focus on upgrade of DatabricksSqlHook
  • Migrate common-sql to 1.27.0
  • Add unit test for pandas and polars

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

@guan404ming guan404ming force-pushed the migrate-databricksql branch from 531c7a8 to c3a3d6b Compare May 13, 2025 06:53
@eladkal
Copy link
Contributor

eladkal commented May 13, 2025

static checks failures are not related will merge once other tests are finished

@guan404ming
Copy link
Member Author

BTW should we also need to upgrade all remaining providers using DBApiHook to new version of common-sql?

@guan404ming
Copy link
Member Author

static checks failures are not related will merge once other tests are finished

Thanks!

@eladkal eladkal merged commit 1794420 into apache:main May 13, 2025
65 of 66 checks passed
@guan404ming guan404ming deleted the migrate-databricksql branch May 13, 2025 07:29
sanederchik pushed a commit to sanederchik/airflow that referenced this pull request Jun 7, 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.

2 participants