snazy commented on code in PR #896: URL: https://github.com/apache/polaris/pull/896#discussion_r1931755271
########## 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 Review Comment: Java 17 WFM. ########## 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: Would 3.12 or 3.12 work? Those versions still get bugfixes (not just security fixes). ########## 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 Review Comment: Q: Is this Scala 2.13? I'd assume so, because there are separate images that have "scala2.12" in their tag name - but no images with "scala2.13". -- 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
