Skip to content

Commit

Permalink
Remove sqlalchemy-redshift dependency from Amazon provider
Browse files Browse the repository at this point in the history
`sqlalchemy-redshift` is unused. It is also not compatible with sqlalchemy>2, so good riddance!
  • Loading branch information
ashb authored and mobuchowski committed Oct 21, 2024
1 parent 892b2f1 commit db3ddbb
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion Dockerfile.ci
Original file line number Diff line number Diff line change
Expand Up @@ -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} \
Expand Down
2 changes: 1 addition & 1 deletion dev/breeze/src/airflow_breeze/global_constants.py
Original file line number Diff line number Diff line change
Expand Up @@ -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]]] = [
Expand Down
1 change: 0 additions & 1 deletion generated/provider_dependencies.json
Original file line number Diff line number Diff line change
Expand Up @@ -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": [
Expand Down
1 change: 0 additions & 1 deletion providers/src/airflow/providers/amazon/provider.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit db3ddbb

Please sign in to comment.