8bitreid commented on a change in pull request #32436:
URL: https://github.com/apache/spark/pull/32436#discussion_r626640992
##########
File path:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodsLifecycleManager.scala
##########
@@ -217,14 +217,42 @@ private[spark] class ExecutorPodsLifecycleManager(
ExecutorExited(exitCode, exitCausedByApp, exitMessage)
}
+ // A utility function to try and help people figure out whats gone wrong
faster.
+ private def describeExitCode(code: Int): String = {
+ val humanStr = code match {
+ case 0 => "(success)"
+ case 1 => "(generic, look at logs to clarify)"
+ case 42 => "(douglas adams)"
+ // Spark specific
+ case 10 => "(Uncaught exception)"
+ case 50 => "(Uncaught exception)"
+ case 52 => "(JVM OOM)"
+ case 53 => "(DiskStore failed to create temp dir)"
+ // K8s & JVM specific exit codes
+ case 126 => "(not executable - possibly perm or arch)"
+ case 137 => "(SIGKILL, possible container OOM)"
+ case 139 => "(SIGSEGV: that's unexpected)"
+ case 255 => "(exit-1, your guess is as good as mine)"
+ case _ => ""
Review comment:
I guess this would be "unexpected" right?
--
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
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
