From 9674af5f88929f5057158241330f4ef4fd08beb2 Mon Sep 17 00:00:00 2001
From: Elad Kalif <45845474+eladkal@users.noreply.github.com>
Date: Thu, 22 Aug 2024 18:01:58 +0300
Subject: [PATCH] Mark tabular provider as removed (#41629)
* Mark tabular provider as removed
---
airflow/providers/tabular/provider.yaml | 2 +-
dev/breeze/doc/images/output-commands.svg | 42 +++++++++----------
...ement_generate-issue-content-providers.svg | 12 +++---
...ement_generate-issue-content-providers.txt | 2 +-
...agement_prepare-provider-documentation.svg | 26 ++++++------
...agement_prepare-provider-documentation.txt | 2 +-
...e-management_prepare-provider-packages.svg | 28 ++++++-------
...e-management_prepare-provider-packages.txt | 2 +-
dev/breeze/tests/test_packages.py | 2 +-
generated/provider_dependencies.json | 2 +-
10 files changed, 60 insertions(+), 60 deletions(-)
diff --git a/airflow/providers/tabular/provider.yaml b/airflow/providers/tabular/provider.yaml
index 2e0530313d649..8ea1c559f0f14 100644
--- a/airflow/providers/tabular/provider.yaml
+++ b/airflow/providers/tabular/provider.yaml
@@ -21,7 +21,7 @@ name: Tabular
description: |
`Tabular `__
-state: ready
+state: removed
source-date-epoch: 1723970601
# note that those versions are maintained by release manager - do not update them manually
versions:
diff --git a/dev/breeze/doc/images/output-commands.svg b/dev/breeze/doc/images/output-commands.svg
index 08d3dc2a13eea..5888d1fc862eb 100644
--- a/dev/breeze/doc/images/output-commands.svg
+++ b/dev/breeze/doc/images/output-commands.svg
@@ -298,53 +298,53 @@
Usage:breeze[OPTIONS] COMMAND [ARGS]...
╭─ Execution mode ─────────────────────────────────────────────────────────────────────────────────────────────────────╮
-│--python-pPython major/minor version used in Airflow image for images.│
+│--python-pPython major/minor version used in Airflow image for images.│
│(>3.8< | 3.9 | 3.10 | 3.11 | 3.12) │
│[default: 3.8] │
-│--integrationIntegration(s) to enable when running (can be more than one). │
+│--integrationIntegration(s) to enable when running (can be more than one). │
│(all | all-testable | cassandra | celery | drill | kafka | kerberos | mongo | mssql │
│| openlineage | otel | pinot | qdrant | redis | statsd | trino | ydb) │
-│--standalone-dag-processorRun standalone dag processor for start-airflow.│
-│--database-isolationRun airflow in database isolation mode.│
+│--standalone-dag-processorRun standalone dag processor for start-airflow.│
+│--database-isolationRun airflow in database isolation mode.│
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
╭─ Docker Compose selection and cleanup ───────────────────────────────────────────────────────────────────────────────╮
-│--project-nameName of the docker-compose project to bring down. The `docker-compose` is for legacy breeze │
-│project name and you can use `breeze down --project-name docker-compose` to stop all containers │
+│--project-nameName of the docker-compose project to bring down. The `docker-compose` is for legacy breeze │
+│project name and you can use `breeze down --project-name docker-compose` to stop all containers │
│belonging to it. │
│(breeze | pre-commit | docker-compose) │
│[default: breeze] │
-│--docker-hostOptional - docker host to use when running docker commands. When set, the `--builder` option is │
+│--docker-hostOptional - docker host to use when running docker commands. When set, the `--builder` option is │
│ignored when building images. │
│(TEXT) │
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
╭─ Database ───────────────────────────────────────────────────────────────────────────────────────────────────────────╮
-│--backend-bDatabase backend to use. If 'none' is chosen, Breeze will start with an invalid database │
+│--backend-bDatabase backend to use. If 'none' is chosen, Breeze will start with an invalid database │
│configuration, meaning there will be no database available, and any attempts to connect to │
│the Airflow database will fail. │
│(>sqlite< | mysql | postgres | none) │
│[default: sqlite] │
-│--postgres-version-PVersion of Postgres used.(>12< | 13 | 14 | 15 | 16)[default: 12]│
-│--mysql-version-MVersion of MySQL used.(>8.0< | 8.4)[default: 8.0]│
-│--db-reset-dReset DB when entering the container.│
+│--postgres-version-PVersion of Postgres used.(>12< | 13 | 14 | 15 | 16)[default: 12]│
+│--mysql-version-MVersion of MySQL used.(>8.0< | 8.4)[default: 8.0]│
+│--db-reset-dReset DB when entering the container.│
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
╭─ Build CI image (before entering shell) ─────────────────────────────────────────────────────────────────────────────╮
-│--github-repository-gGitHub repository used to pull, push run images.(TEXT)[default: apache/airflow]│
-│--builderBuildx builder used to perform `docker buildx build` commands.(TEXT)│
+│--github-repository-gGitHub repository used to pull, push run images.(TEXT)[default: apache/airflow]│
+│--builderBuildx builder used to perform `docker buildx build` commands.(TEXT)│
│[default: autodetect] │
-│--use-uv/--no-use-uvUse uv instead of pip as packaging tool to build the image.[default: use-uv]│
-│--uv-http-timeoutTimeout for requests that UV makes (only used in case of UV builds).(INTEGER RANGE)│
+│--use-uv/--no-use-uvUse uv instead of pip as packaging tool to build the image.[default: use-uv]│
+│--uv-http-timeoutTimeout for requests that UV makes (only used in case of UV builds).(INTEGER RANGE)│
│[default: 300; x>=1] │
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
╭─ Other options ──────────────────────────────────────────────────────────────────────────────────────────────────────╮
-│--forward-credentials-fForward local credentials to container when running.│
-│--max-timeMaximum time that the command should take - if it takes longer, the command will fail.│
+│--forward-credentials-fForward local credentials to container when running.│
+│--max-timeMaximum time that the command should take - if it takes longer, the command will fail.│
│(INTEGER RANGE) │
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
╭─ Common options ─────────────────────────────────────────────────────────────────────────────────────────────────────╮
-│--answer-aForce answer to questions.(y | n | q | yes | no | quit)│
-│--dry-run-DIf dry-run is set, commands are only printed, not executed.│
-│--verbose-vPrint verbose information about performed steps.│
-│--help-hShow this message and exit.│
+│--answer-aForce answer to questions.(y | n | q | yes | no | quit)│
+│--dry-run-DIf dry-run is set, commands are only printed, not executed.│
+│--verbose-vPrint verbose information about performed steps.│
+│--help-hShow this message and exit.│
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
╭─ Developer commands ─────────────────────────────────────────────────────────────────────────────────────────────────╮
│start-airflow Enter breeze environment and starts all Airflow components in the tmux session. Compile │
diff --git a/dev/breeze/doc/images/output_release-management_generate-issue-content-providers.svg b/dev/breeze/doc/images/output_release-management_generate-issue-content-providers.svg
index 0c75541c88d3f..62ed9b25dda68 100644
--- a/dev/breeze/doc/images/output_release-management_generate-issue-content-providers.svg
+++ b/dev/breeze/doc/images/output_release-management_generate-issue-content-providers.svg
@@ -146,21 +146,21 @@
jenkins | microsoft.azure | microsoft.mssql | microsoft.psrp | microsoft.winrm | mongo | mysql | neo4j | odbc | openai
| openfaas | openlineage | opensearch | opsgenie | oracle | pagerduty | papermill | pgvector | pinecone | postgres |
presto | qdrant | redis | salesforce | samba | segment | sendgrid | sftp | singularity | slack | smtp | snowflake |
-sqlite | ssh | tableau | tabular | telegram | teradata | trino | vertica | weaviate | yandex | ydb | zendesk]...
+sqlite | ssh | tableau | telegram | teradata | trino | vertica | weaviate | yandex | ydb | zendesk]...
Generates content for issue to test the release.
╭─ Generate issue content flags ───────────────────────────────────────────────────────────────────────────────────────╮
-│--disable-progressDisable progress bar│
-│--excluded-pr-listComa-separated list of PRs to exclude from the issue.(TEXT)│
-│--github-tokenGitHub token used to authenticate. You can set omit it if you have GITHUB_TOKEN env │
+│--disable-progressDisable progress bar│
+│--excluded-pr-listComa-separated list of PRs to exclude from the issue.(TEXT)│
+│--github-tokenGitHub token used to authenticate. You can set omit it if you have GITHUB_TOKEN env │
│variable set. Can be generated with: │
│https://github.com/settings/tokens/new?description=Read%20sssues&scopes=repo:status │
│(TEXT) │
-│--only-available-in-distOnly consider package ids with packages prepared in the dist folder│
+│--only-available-in-distOnly consider package ids with packages prepared in the dist folder│
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
╭─ Common options ─────────────────────────────────────────────────────────────────────────────────────────────────────╮
-│--help-hShow this message and exit.│
+│--help-hShow this message and exit.│
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
diff --git a/dev/breeze/doc/images/output_release-management_generate-issue-content-providers.txt b/dev/breeze/doc/images/output_release-management_generate-issue-content-providers.txt
index 727744ffc0d22..db1315caea441 100644
--- a/dev/breeze/doc/images/output_release-management_generate-issue-content-providers.txt
+++ b/dev/breeze/doc/images/output_release-management_generate-issue-content-providers.txt
@@ -1 +1 @@
-4d19bd6dfbf0e76b1e853760cb02cab5
+0aa10fcd2a9fcdaf15f0b207d71e3848
diff --git a/dev/breeze/doc/images/output_release-management_prepare-provider-documentation.svg b/dev/breeze/doc/images/output_release-management_prepare-provider-documentation.svg
index 0045a74e1e5ce..6bf07f9bc7f3d 100644
--- a/dev/breeze/doc/images/output_release-management_prepare-provider-documentation.svg
+++ b/dev/breeze/doc/images/output_release-management_prepare-provider-documentation.svg
@@ -182,33 +182,33 @@
jenkins | microsoft.azure | microsoft.mssql | microsoft.psrp | microsoft.winrm | mongo | mysql | neo4j | odbc | openai
| openfaas | openlineage | opensearch | opsgenie | oracle | pagerduty | papermill | pgvector | pinecone | postgres |
presto | qdrant | redis | salesforce | samba | segment | sendgrid | sftp | singularity | slack | smtp | snowflake |
-sqlite | ssh | tableau | tabular | telegram | teradata | trino | vertica | weaviate | yandex | ydb | zendesk]...
+sqlite | ssh | tableau | telegram | teradata | trino | vertica | weaviate | yandex | ydb | zendesk]...
Prepare CHANGELOG, README and COMMITS information for providers.
╭─ Provider documentation preparation flags ───────────────────────────────────────────────────────────────────────────╮
-│--base-branchBase branch to use as diff for documentation generation (used for releasing from │
+│--base-branchBase branch to use as diff for documentation generation (used for releasing from │
│old branch) │
│(TEXT) │
-│--github-repository-gGitHub repository used to pull, push run images.(TEXT)[default: apache/airflow]│
-│--include-not-ready-providersWhether to include providers that are not yet ready to be released.│
-│--include-removed-providersWhether to include providers that are removed.│
-│--non-interactiveRun in non-interactive mode. Provides random answers to the type of changes and │
+│--github-repository-gGitHub repository used to pull, push run images.(TEXT)[default: apache/airflow]│
+│--include-not-ready-providersWhether to include providers that are not yet ready to be released.│
+│--include-removed-providersWhether to include providers that are removed.│
+│--non-interactiveRun in non-interactive mode. Provides random answers to the type of changes and │
│confirms releasefor providers prepared for release - useful to test the script in│
│non-interactive mode in CI. │
-│--only-min-version-updateOnly update minimum version in __init__.py files and regenerate corresponding │
+│--only-min-version-updateOnly update minimum version in __init__.py files and regenerate corresponding │
│documentation │
-│--reapply-templates-onlyOnly reapply templates, do not bump version. Useful if templates were added and │
+│--reapply-templates-onlyOnly reapply templates, do not bump version. Useful if templates were added and │
│you need to regenerate documentation. │
-│--skip-git-fetchSkips removal and recreation of `apache-https-for-providers` remote in git. By │
+│--skip-git-fetchSkips removal and recreation of `apache-https-for-providers` remote in git. By │
│default, the remote is recreated and fetched to make sure that it's up to date │
│and that recent commits are not missing │
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
╭─ Common options ─────────────────────────────────────────────────────────────────────────────────────────────────────╮
-│--answer-aForce answer to questions.(y | n | q | yes | no | quit)│
-│--dry-run-DIf dry-run is set, commands are only printed, not executed.│
-│--verbose-vPrint verbose information about performed steps.│
-│--help-hShow this message and exit.│
+│--answer-aForce answer to questions.(y | n | q | yes | no | quit)│
+│--dry-run-DIf dry-run is set, commands are only printed, not executed.│
+│--verbose-vPrint verbose information about performed steps.│
+│--help-hShow this message and exit.│
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
diff --git a/dev/breeze/doc/images/output_release-management_prepare-provider-documentation.txt b/dev/breeze/doc/images/output_release-management_prepare-provider-documentation.txt
index 14192f1e6cc8e..20e2e14faefed 100644
--- a/dev/breeze/doc/images/output_release-management_prepare-provider-documentation.txt
+++ b/dev/breeze/doc/images/output_release-management_prepare-provider-documentation.txt
@@ -1 +1 @@
-2712510aa5a989444d87e66f8f896e80
+6953cb3aa552b67616179e1acfe7e8c3
diff --git a/dev/breeze/doc/images/output_release-management_prepare-provider-packages.svg b/dev/breeze/doc/images/output_release-management_prepare-provider-packages.svg
index eba524437dc4c..3c4e468aba0c7 100644
--- a/dev/breeze/doc/images/output_release-management_prepare-provider-packages.svg
+++ b/dev/breeze/doc/images/output_release-management_prepare-provider-packages.svg
@@ -182,33 +182,33 @@
jenkins | microsoft.azure | microsoft.mssql | microsoft.psrp | microsoft.winrm | mongo | mysql | neo4j | odbc | openai
| openfaas | openlineage | opensearch | opsgenie | oracle | pagerduty | papermill | pgvector | pinecone | postgres |
presto | qdrant | redis | salesforce | samba | segment | sendgrid | sftp | singularity | slack | smtp | snowflake |
-sqlite | ssh | tableau | tabular | telegram | teradata | trino | vertica | weaviate | yandex | ydb | zendesk]...
+sqlite | ssh | tableau | telegram | teradata | trino | vertica | weaviate | yandex | ydb | zendesk]...
Prepare sdist/whl packages of Airflow Providers.
╭─ Package flags ──────────────────────────────────────────────────────────────────────────────────────────────────────╮
-│--clean-distClean dist directory before building packages. Useful when you want to build │
+│--clean-distClean dist directory before building packages. Useful when you want to build │
│multiple packages in a clean environment │
-│--github-repository-gGitHub repository used to pull, push run images.(TEXT)│
+│--github-repository-gGitHub repository used to pull, push run images.(TEXT)│
│[default: apache/airflow] │
-│--include-not-ready-providersWhether to include providers that are not yet ready to be released.│
-│--include-removed-providersWhether to include providers that are removed.│
-│--package-formatFormat of packages.(wheel | sdist | both)[default: wheel]│
-│--package-list-fileRead list of packages from text file (one package per line).(FILENAME)│
-│--skip-deleting-generated-filesSkip deleting files that were used to generate provider package. Useful for │
+│--include-not-ready-providersWhether to include providers that are not yet ready to be released.│
+│--include-removed-providersWhether to include providers that are removed.│
+│--package-formatFormat of packages.(wheel | sdist | both)[default: wheel]│
+│--package-list-fileRead list of packages from text file (one package per line).(FILENAME)│
+│--skip-deleting-generated-filesSkip deleting files that were used to generate provider package. Useful for │
│debugging and developing changes to the build process. │
-│--skip-tag-checkSkip checking if the tag already exists in the remote repository│
-│--version-suffix-for-pypiVersion suffix used for PyPI packages (alpha, beta, rc1, etc.).(TEXT)│
-│--package-listOptional, contains comma-separated list of package ids that are processed for │
+│--skip-tag-checkSkip checking if the tag already exists in the remote repository│
+│--version-suffix-for-pypiVersion suffix used for PyPI packages (alpha, beta, rc1, etc.).(TEXT)│
+│--package-listOptional, contains comma-separated list of package ids that are processed for │
│documentation building, and document publishing. It is an easier alternative to│
│adding individual packages as arguments to every command. This overrides the │
│packages passed as arguments. │
│(TEXT) │
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
╭─ Common options ─────────────────────────────────────────────────────────────────────────────────────────────────────╮
-│--dry-run-DIf dry-run is set, commands are only printed, not executed.│
-│--verbose-vPrint verbose information about performed steps.│
-│--help-hShow this message and exit.│
+│--dry-run-DIf dry-run is set, commands are only printed, not executed.│
+│--verbose-vPrint verbose information about performed steps.│
+│--help-hShow this message and exit.│
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
diff --git a/dev/breeze/doc/images/output_release-management_prepare-provider-packages.txt b/dev/breeze/doc/images/output_release-management_prepare-provider-packages.txt
index b8876b08183de..c40492220b2a0 100644
--- a/dev/breeze/doc/images/output_release-management_prepare-provider-packages.txt
+++ b/dev/breeze/doc/images/output_release-management_prepare-provider-packages.txt
@@ -1 +1 @@
-96b181d19e0f03f87f2927e1946625ab
+b3d53d1c7e5233cebb3e35afed6ed8e6
diff --git a/dev/breeze/tests/test_packages.py b/dev/breeze/tests/test_packages.py
index 3294c197e5849..873a4212e122d 100644
--- a/dev/breeze/tests/test_packages.py
+++ b/dev/breeze/tests/test_packages.py
@@ -109,7 +109,7 @@ def test_get_provider_requirements():
def test_get_removed_providers():
# Modify it every time we schedule provider for removal or remove it
- assert [] == get_removed_provider_ids()
+ assert ["tabular"] == get_removed_provider_ids()
def test_get_suspended_provider_ids():
diff --git a/generated/provider_dependencies.json b/generated/provider_dependencies.json
index ce59ba6127b13..683f72e41efc6 100644
--- a/generated/provider_dependencies.json
+++ b/generated/provider_dependencies.json
@@ -1271,7 +1271,7 @@
"apache.iceberg"
],
"excluded-python-versions": [],
- "state": "ready"
+ "state": "removed"
},
"telegram": {
"deps": [