tvalentyn commented on code in PR #37822:
URL: https://github.com/apache/beam/pull/37822#discussion_r2931236222


##########
sdks/python/setup.py:
##########
@@ -542,6 +540,7 @@ def get_portability_package_data():
               # hard requirement in setup.py.
               'dill',
               'tensorflow-transform',
+              'protobuf<4; python_version<"3.11"',

Review Comment:
   Note: without this change, 
   
   
[beam_PreCommit_Yaml_Xlang_Direct](https://github.com/apache/beam/actions/workflows/beam_PreCommit_Yaml_Xlang_Direct.yml)
 suite failed to resolve dependencies. Looking at successful logs 
(https://github.com/apache/beam/actions/runs/23049196161/job/66945823677) the 
suite already installed protobuf 3:
   
   ```
   2026-03-13T11:58:14.2393149Z Successfully installed absl-py-1.4.0 
annotated-doc-0.0.4 apache-beam-2.73.0.dev0 astunparse-1.6.3 click-8.3.1 
coloredlogs-15.0.1 contourpy-1.3.2 cuda-bindings-12.9.4 cuda-pathfinder-1.4.2 
cycler-0.12.1 datatable-1.1.0 dill-0.4.1 embeddings-0.0.8 flatbuffers-25.12.19 
fonttools-4.62.0 fsspec-2026.2.0 gast-0.7.0 google-api-core-2.29.0 
google-api-python-client-1.12.11 google_pasta-0.2.0 grpcio-status-1.49.0rc1 
h5py-3.16.0 hf-xet-1.4.2 huggingface-hub-1.7.1 humanfriendly-10.0 js2py-0.74 
jsonpatch-1.33 jsonpointer-3.0.0 jsonschema-4.26.0 
jsonschema-specifications-2025.9.1 keras-3.12.1 kiwisolver-1.5.0 
langchain-1.2.12 langchain-core-1.2.18 langgraph-1.1.2 
langgraph-checkpoint-4.0.1 langgraph-prebuilt-1.0.8 langgraph-sdk-0.3.11 
langsmith-0.7.17 libclang-18.1.1 llvmlite-0.46.0 markdown-3.10.2 
markdown-it-py-4.0.0 matplotlib-3.10.8 mdurl-0.1.2 ml-dtypes-0.3.2 mpmath-1.3.0 
namex-0.1.0 networkx-3.4.2 numba-0.64.0 numpy-1.26.4 
nvidia-cublas-cu12-12.8.4.1 nvidia-cu
 da-cupti-cu12-12.8.90 nvidia-cuda-nvrtc-cu12-12.8.93 
nvidia-cuda-runtime-cu12-12.8.90 nvidia-cudnn-cu12-9.10.2.21 
nvidia-cufft-cu12-11.3.3.83 nvidia-cufile-cu12-1.13.1.3 
nvidia-curand-cu12-10.3.9.90 nvidia-cusolver-cu12-11.7.3.90 
nvidia-cusparse-cu12-12.5.8.93 nvidia-cusparselt-cu12-0.7.1 
nvidia-nccl-cu12-2.27.5 nvidia-nvjitlink-cu12-12.8.93 nvidia-nvshmem-cu12-3.4.5 
nvidia-nvtx-cu12-12.8.90 onnx-1.17.0 onnxruntime-1.23.2 opt_einsum-3.4.0 
optree-0.19.0 ormsgpack-1.12.2 pandas-1.5.3 protobuf-3.20.3 pyarrow-10.0.1 
pydot-1.4.2 pyjsparser-2.7.1 pyod-2.0.7 referencing-0.37.0 
requests-toolbelt-1.0.0 rich-14.3.3 rpds-py-0.30.0 safetensors-0.7.0 
sentence-transformers-5.3.0 shellingham-1.5.4 skl2onnx-1.20.0 sympy-1.14.0 
tensorboard-2.16.2 tensorboard-data-server-0.7.2 tensorflow-2.16.2 
tensorflow-hub-0.16.1 tensorflow-io-gcs-filesystem-0.37.1 
tensorflow-metadata-1.16.1 tensorflow-serving-api-2.16.1 
tensorflow-transform-1.16.0 termcolor-3.3.0 tf-keras-2.16.0 tf2onnx-1.17.0 
tfx-bsl-1.16.1 toke
 nizers-0.22.2 torch-2.10.0 tqdm-4.67.3 transformers-5.3.0 triton-3.6.0 
typer-0.24.1 tzlocal-5.3.1 uritemplate-3.0.1 uuid-utils-0.14.1 werkzeug-3.1.6 
wheel-0.46.3 xxhash-3.6.0
   2026-03-13T11:58:22.8382660Z 
   2026-03-13T11:58:22.8383507Z > Task :sdks:python:yamlIntegrationTests
   ```



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