MonkeyCanCode commented on code in PR #896:
URL: https://github.com/apache/polaris/pull/896#discussion_r1932170609


##########
regtests/Dockerfile:
##########
@@ -17,14 +17,14 @@
 # under the License.
 #
 
-FROM docker.io/apache/spark:3.5.4-python3
+FROM docker.io/apache/spark:3.5.4-java17-python3
 ARG POLARIS_HOST=polaris
 ENV POLARIS_HOST=$POLARIS_HOST
 ENV SPARK_HOME=/opt/spark
 
 USER root
 RUN apt update
-RUN apt-get install -y diffutils wget curl python3.8-venv
+RUN apt-get install -y diffutils wget curl python3.10-venv

Review Comment:
   It will. For my local, I am using 3.13. However, if we want to use the 
official spark image and different versions of python, we will need to compile 
from source code. This, in my previous PR of rework the test cases, I was using 
Python as a base image and built our own spark image. Both will work. 



-- 
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: issues-unsubscr...@polaris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to