[GitHub] [spark] HyukjinKwon commented on a diff in pull request #43076: [SPARK-45276][INFRA] Replace Java 8 and Java 11 installed in the Dockerfile with Java

["HyukjinKwon (via GitHub)" <gi...@apache.org>]

HyukjinKwon commented on code in PR #43076:
URL: https://github.com/apache/spark/pull/43076#discussion_r1335295623


##########
dev/create-release/spark-rm/Dockerfile:
##########
@@ -61,9 +61,9 @@ RUN apt-get clean && apt-get update && $APT_INSTALL gnupg ca-certificates && \
   apt-get update && \

Review Comment:
   Since you're here, can you also fix the comments at `dev/create-release/do-release-docker.sh`:
   
   ```
     -j [path]   : path to local JDK installation to use for building. By default the script will
                   use openjdk8 installed in the docker image.
   ```
   



-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org