From db3ddbb270d0e75f15ad2195ebbb2ad1711bfe6f Mon Sep 17 00:00:00 2001 From: Ash Berlin-Taylor Date: Sat, 12 Oct 2024 17:32:24 +0100 Subject: [PATCH] Remove sqlalchemy-redshift dependency from Amazon provider `sqlalchemy-redshift` is unused. It is also not compatible with sqlalchemy>2, so good riddance! --- Dockerfile.ci | 2 +- dev/breeze/src/airflow_breeze/global_constants.py | 2 +- generated/provider_dependencies.json | 1 - providers/src/airflow/providers/amazon/provider.yaml | 1 - 4 files changed, 2 insertions(+), 4 deletions(-) diff --git a/Dockerfile.ci b/Dockerfile.ci index 3ddba289a807f..cdf80c9b91593 100644 --- a/Dockerfile.ci +++ b/Dockerfile.ci @@ -1195,7 +1195,7 @@ ARG AIRFLOW_IMAGE_REPOSITORY="https://github.com/apache/airflow" # NOTE! When you want to make sure dependencies are installed from scratch in your PR after removing # some dependencies, you also need to set "disable image cache" in your PR to make sure the image is # not built using the "main" version of those dependencies. -ARG DEPENDENCIES_EPOCH_NUMBER="11" +ARG DEPENDENCIES_EPOCH_NUMBER="12" # Make sure noninteractive debian install is used and language variables set ENV PYTHON_BASE_IMAGE=${PYTHON_BASE_IMAGE} \ diff --git a/dev/breeze/src/airflow_breeze/global_constants.py b/dev/breeze/src/airflow_breeze/global_constants.py index 97001746409ac..ebf235b2cb4e3 100644 --- a/dev/breeze/src/airflow_breeze/global_constants.py +++ b/dev/breeze/src/airflow_breeze/global_constants.py @@ -534,7 +534,7 @@ def get_airflow_extras(): # END OF EXTRAS LIST UPDATED BY PRE COMMIT ] -CHICKEN_EGG_PROVIDERS = " ".join(["standard"]) +CHICKEN_EGG_PROVIDERS = " ".join(["standard amazon"]) BASE_PROVIDERS_COMPATIBILITY_CHECKS: list[dict[str, str | list[str]]] = [ diff --git a/generated/provider_dependencies.json b/generated/provider_dependencies.json index 4457556be0da4..0e39f058afe2d 100644 --- a/generated/provider_dependencies.json +++ b/generated/provider_dependencies.json @@ -38,7 +38,6 @@ "jsonpath_ng>=1.5.3", "python3-saml>=1.16.0", "redshift_connector>=2.0.918", - "sqlalchemy_redshift>=0.8.6", "watchtower>=3.0.0,!=3.3.0,<4" ], "devel-deps": [ diff --git a/providers/src/airflow/providers/amazon/provider.yaml b/providers/src/airflow/providers/amazon/provider.yaml index a8d182b904954..11e6dc8db753f 100644 --- a/providers/src/airflow/providers/amazon/provider.yaml +++ b/providers/src/airflow/providers/amazon/provider.yaml @@ -106,7 +106,6 @@ dependencies: - watchtower>=3.0.0,!=3.3.0,<4 - jsonpath_ng>=1.5.3 - redshift_connector>=2.0.918 - - sqlalchemy_redshift>=0.8.6 - asgiref>=2.3.0 - PyAthena>=3.0.10 - jmespath>=0.7.0