Skip to content

Commit e7c97a3

Browse files
onursaticisrowen
authored andcommitted
[SPARK-28145][K8S] safe runnable in polling executor source
## What changes were proposed in this pull request? Add error handling to `ExecutorPodsPollingSnapshotSource` Closes apache#24952 from onursatici/os/polling-source. Authored-by: Onur Satici <[email protected]> Signed-off-by: Sean Owen <[email protected]>
1 parent 832ff87 commit e7c97a3

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodsPollingSnapshotSource.scala

+2-2
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ import org.apache.spark.SparkConf
2525
import org.apache.spark.deploy.k8s.Config._
2626
import org.apache.spark.deploy.k8s.Constants._
2727
import org.apache.spark.internal.Logging
28-
import org.apache.spark.util.ThreadUtils
28+
import org.apache.spark.util.{ThreadUtils, Utils}
2929

3030
private[spark] class ExecutorPodsPollingSnapshotSource(
3131
conf: SparkConf,
@@ -53,7 +53,7 @@ private[spark] class ExecutorPodsPollingSnapshotSource(
5353
}
5454

5555
private class PollRunnable(applicationId: String) extends Runnable {
56-
override def run(): Unit = {
56+
override def run(): Unit = Utils.tryLogNonFatalError {
5757
logDebug(s"Resynchronizing full executor pod state from Kubernetes.")
5858
snapshotsStore.replaceSnapshot(kubernetesClient
5959
.pods()

0 commit comments

Comments
 (0)