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


##########
sdks/python/container/Dockerfile:
##########
@@ -43,9 +40,12 @@ RUN  \
        # Required for using Beam Python SDK on ARM machines.
        libgeos-dev \
        && \
-    rm -rf /var/lib/apt/lists/* && \
+    rm -rf /var/lib/apt/lists/*
 
-    pip install --upgrade pip setuptools wheel && \
+RUN python -m venv venv-beam

Review Comment:
   Beam SDK workers create a venv as well, which is superimposed upon the 
default environment. I think having Beam dependencies packages in a separate 
venv would break this setup.



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