wangyum commented on a change in pull request #26848: [SPARK-30216][INFRA] Use 
python3 in Docker release image
URL: https://github.com/apache/spark/pull/26848#discussion_r357000600
 
 

 ##########
 File path: dev/create-release/spark-rm/Dockerfile
 ##########
 @@ -62,14 +62,18 @@ RUN apt-get clean && apt-get update && $APT_INSTALL gnupg 
ca-certificates && \
   curl -sL https://deb.nodesource.com/setup_11.x | bash && \
   $APT_INSTALL nodejs && \
   # Install needed python packages. Use pip for installing packages (for 
consistency).
-  $APT_INSTALL libpython2.7-dev libpython3-dev python-pip python3-pip && \
-  pip install $BASE_PIP_PKGS && \
-  pip install $PIP_PKGS && \
+  $APT_INSTALL libpython3-dev python3-pip && \
+  # Change default python version to python3.
+  update-alternatives --install /usr/bin/python python /usr/bin/python2.7 1 && 
\
+  update-alternatives --install /usr/bin/python python /usr/bin/python3.6 2 && 
\
+  update-alternatives --set python /usr/bin/python3.6 && \
+  pip3 install $BASE_PIP_PKGS && \
+  pip3 install $PIP_PKGS && \
   cd && \
-  virtualenv -p python3 /opt/p35 && \
+  python /opt/p35 && \
 
 Review comment:
   I reverted commit 1f94bf4 and d6be46e, I'd like to to confirm is it correct??
   
![image](https://user-images.githubusercontent.com/5399861/70693075-a7723100-1cf7-11ea-9f88-9356a02349a1.png)
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to