airflow icon indicating copy to clipboard operation
airflow copied to clipboard

Add TabularVttsSensor

Open Fokko opened this issue 1 year ago • 4 comments

Second try of https://github.com/apache/airflow/pull/32786


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

Fokko avatar Apr 17 '24 15:04 Fokko

Do we want to add this before the creation of iceberg provider ? because it means adding class that we will deprecate (soon?) https://github.com/apache/airflow/pull/32786#issuecomment-1768284013 I think @romsharon98 expressed interest in handling the migration from tabular provider to iceberg provider

eladkal avatar Apr 17 '24 17:04 eladkal

@eladkal Yes, it is called the TabularVttsSensor, but it doesn't have anything Tabular specific, it is fully Iceberg OSS compatible.

Fokko avatar Apr 17 '24 19:04 Fokko

@Fokko We can now resume this PR on the iceberg provider rather than tabular

eladkal avatar May 12 '24 19:05 eladkal

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Jun 29 '24 00:06 github-actions[bot]