holdenk commented on a change in pull request #33508:
URL: https://github.com/apache/spark/pull/33508#discussion_r682913691
##########
File path:
resource-managers/kubernetes/docker/src/main/dockerfiles/spark/entrypoint.sh
##########
@@ -79,6 +79,14 @@ case "$1" in
;;
executor)
shift 1
+ # If the execid is set to the generic EXECID parse the pod name
+ if [ "${SPARK_EXECUTOR_ID}" == "EXECID" ]; then
Review comment:
That didn't seem to be a good match since our executor snapshot depends
on being able to determine the exec id for a given pod as well and if we did it
fully dynamically we wouldn't be able to know and we'd have to make even bigger
changes to how we handle snapshot updates.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
