uros-b commented on code in PR #57233:
URL: https://github.com/apache/spark/pull/57233#discussion_r3614051018
##########
pyproject.toml:
##########
@@ -198,3 +212,49 @@ dev = [
{include-group = "pandas_on_spark_extra"},
{include-group = "pipelines"},
]
+
+_ci_connect = [
+ {include-group = "connect"},
+ "grpcio==1.76.0",
+ "grpcio-status==1.76.0",
+ "googleapis-common-protos==1.71.0",
+ "zstandard==0.25.0",
+ "protobuf==6.33.5",
+]
+
+ci_connect_standard = [
+ {include-group = "_ci_connect"},
+ "graphviz==0.20.3",
+]
+
+ci_connect_minimum = [
Review Comment:
Is this internally unresolvable? It pulls in connect (which pins
graphviz==0.20.3) via _ci_connect, then also lists graphviz==0.20 directly, so
pip install --group ci_connect_minimum combines graphviz==0.20.3,==0.20 and
cannot resolve. Latent/non-blocking (no Dockerfile references
ci_connect_minimum at head; it is staged for a later python-minimum
conversion), but the group as written will fail whoever wires it up, and the
apparent intent (loosen graphviz for the minimum image) is defeated by
connect's exact pin. Likely should be fixed to ==0.20.3 (or factor the graphviz
pin out of connect).
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
