tech4242 commented on issue #27987: [SPARK-31165] Correcting wrong paths in 
Dockerfile
URL: https://github.com/apache/spark/pull/27987#issuecomment-602670293
 
 
   @srowen @yaooqinn thank you for your feedback! 
   
   I now ran it with `dev/make-distribution.sh -Pkubernetes` instead of 
`./build/mvn -Pkubernetes -DskipTests clean package` and it in fact does create 
a new folder in spark/dist/ where the structure is as described in the 
Dockerfile. 
   
   I propose that we mention this more explicitly in the documentation 
https://spark.apache.org/docs/latest/running-on-kubernetes.html to not run 
`./build/mvn -Pkubernetes -DskipTests clean package` because if you are new-ish 
to the ecosystem you might end up running that command instead of downloading 
the distribution or running the said bash script (regardless of what is 
mentioned in the Prerequisites about the Spark distribution, which I am myself 
guilty of reading over too quickly) 
   
   Thoughts? 

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

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

Reply via email to