potiuk commented on code in PR #40707: URL: https://github.com/apache/airflow/pull/40707#discussion_r1673872826
########## airflow/providers/google/provider.yaml: ########## @@ -130,7 +130,7 @@ dependencies: - google-cloud-dataflow-client>=0.8.6 - google-cloud-dataform>=0.5.0 - google-cloud-dataplex>=1.10.0 - - google-cloud-dataproc>=5.8.0 + - google-cloud-dataproc>=5.10.1 Review Comment: Yeah. Quite agree with Elad here. It might cause some conflicts if we are updating the minimum requirements "too aggressively". Current constraints already point out to 5.10.1 https://github.com/apache/airflow/blob/constraints-main/constraints-3.10.txt#L341 and when people will be installing provider "with constraints" they will get it (they will also get it when installing provider without constraints). We have the "lowest dependencies" check running that currently verifies if we are not using - in our providers, some features that are not available in the "lowest bound" dependency versions - and we will bump minimum version when it finds that it should be bumped. There is no particular reason I think we should bump it manually here. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
