MaksYermak commented on code in PR #66952:
URL: https://github.com/apache/airflow/pull/66952#discussion_r3474031310


##########
providers/apache/beam/pyproject.toml:
##########
@@ -61,7 +61,11 @@ requires-python = ">=3.10"
 dependencies = [
     "apache-airflow>=2.11.0",
     "apache-airflow-providers-common-compat>=1.12.0",
-    "apache-beam>=2.69.0",
+    "apache-beam>=2.72.0",
+    # For Python>=3.11, the apache-beam package depends on envoy-data-plane.
+    # Currently, there is only one compatible version(1.0.3) of 
envoy-data-plane available for these
+    # Python versions, and it explicitly requires a prerelease version of 
betterproto.
+    "betterproto==2.0.0b6; python_version >= '3.11'",

Review Comment:
   I am not a big fan of this idea either.
   
   The reason why I decided to add it was in that this beta version exists from 
2023 and I am not sure that it will be release in the future. One more point 
for me to added it was that I noticed that Apache Beam SDK use this beta 
version in there [generated 
requirements](https://github.com/apache/beam/blob/v2.74.0/sdks/python/container/py311/base_image_requirements.txt#L35)
 for containers. And I had idea that if they already had hardcoded it in 
`apache-beam` requirements, from which our provider already depends, maybe it 
makes sense temporary hardcoded it in provider as well for fixing constrains 
problem and unsuspend provider.  



-- 
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]

Reply via email to