jlpedrosa commented on a change in pull request #24702: [SPARK-27989]
[Kubernetes] [Core] Added retries on the connection to the driver for k8s
URL: https://github.com/apache/spark/pull/24702#discussion_r294316780
##########
File path:
resource-managers/kubernetes/docker/src/main/dockerfiles/spark/entrypoint.sh
##########
@@ -78,6 +78,16 @@ case "$1" in
-Xms$SPARK_EXECUTOR_MEMORY
-Xmx$SPARK_EXECUTOR_MEMORY
-cp "$SPARK_CLASSPATH"
+ )
+
+ #Disable negative dns reslolution
https://docs.oracle.com/javase/8/docs/technotes/guides/net/properties.html if
that property is defined
Review comment:
@srowen @skonto
I am afraid I am not following you. @skonto pointed out that he preferred
the fix instead on Docker file, in the entrypoint.sh script with a configurable
variable (which is also specific for k8s) so this is what this patch has. As
you said "I wouldn't do this here" I though you also meant the Dockerfile and
you agreed.
So I am not sure what do you mean by "wouldn't do this here", where is here
and where is the place where you want it?
----------------------------------------------------------------
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]
