Skip to content

Conversation

@guan404ming
Copy link
Member

@guan404ming guan404ming commented Apr 30, 2025

Related Issue

Why

get_pandas_df deprecated in #48875, which would be replaced by get_df. Thus, we need to migrate them in providers.

How

  • This PR is focus on migration of Exasol provider.
  • Currently polars is not supported see Support for Polars exasol/pyexasol#102, thus I raise NotImplementedError for it.
  • Add missing test for get_df

cc: @eladkal @potiuk


^ 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 changed the title Update Exasol provider dependencies and deprecate get_pandas_df method Migrate Exasol to use get_df Apr 30, 2025
@guan404ming guan404ming changed the title Migrate Exasol to use get_df Migrate Exasol provider to use get_df Apr 30, 2025
@guan404ming guan404ming marked this pull request as draft April 30, 2025 11:12
@guan404ming guan404ming marked this pull request as ready for review April 30, 2025 11:33
@guan404ming guan404ming changed the title Migrate Exasol provider to use get_df refactor: migrate Exasol provider to use get_df Apr 30, 2025
@guan404ming guan404ming changed the title refactor: migrate Exasol provider to use get_df Migrate Exasol provider to use get_df Apr 30, 2025
@potiuk potiuk merged commit c37e6eb into apache:main Apr 30, 2025
64 checks passed
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