Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion composer/2022_airflow_summit/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# be sure to update the constraints file to match
# see https://airflow.apache.org/docs/apache-airflow/stable/installation/installing-from-pypi.html#constraints-files
apache-airflow==2.6.3
apache-airflow-providers-google==10.2.0
apache-airflow-providers-google==18.0.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

critical

While this line correctly updates the provider version, the corresponding constraints.txt file in the same directory has not been updated. It still specifies apache-airflow-providers-google==10.2.0, which will conflict with this change and likely cause dependency resolution to fail. As noted in the comment at the top of this file, the constraints file must be kept in sync.

Additionally, this is a major version upgrade from 10.2.0 to 18.0.0. Such a large jump will almost certainly include breaking changes. The DAGs in this project use operators such as DataprocCreateBatchOperator, GCSToBigQueryOperator, and BigQueryInsertJobOperator. It is crucial to review the provider's changelog and thoroughly test the DAGs to ensure they are compatible with the new version.