[GitHub] [spark] yaooqinn commented on a diff in pull request #38732: [SPARK-41210][K8S] Window based executor failure tracking mechanism

yaooqinn commented on code in PR #38732: URL: https://github.com/apache/spark/pull/38732#discussion_r1152951801 ## resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodsAllocator.scala: ## @@ -494,10 +525,46 @@ class ExecutorPodsAllo

[GitHub] [spark] yaooqinn commented on a diff in pull request #38732: [SPARK-41210][K8S] Window based executor failure tracking mechanism

yaooqinn commented on code in PR #38732: URL: https://github.com/apache/spark/pull/38732#discussion_r1152954378 ## resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodsAllocator.scala: ## @@ -520,10 +552,46 @@ class ExecutorPodsAllo

[GitHub] [spark] yaooqinn commented on a diff in pull request #38732: [SPARK-41210][K8S] Window based executor failure tracking mechanism

yaooqinn commented on code in PR #38732: URL: https://github.com/apache/spark/pull/38732#discussion_r1152957287 ## resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/Config.scala: ## @@ -750,6 +750,26 @@ private[spark] object Config extends Logging {

[GitHub] [spark] yaooqinn commented on a diff in pull request #38732: [SPARK-41210][K8S] Window based executor failure tracking mechanism

yaooqinn commented on code in PR #38732: URL: https://github.com/apache/spark/pull/38732#discussion_r1152960657 ## resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodsAllocator.scala: ## @@ -117,6 +120,12 @@ class ExecutorPodsAlloc

[GitHub] [spark] yaooqinn commented on a diff in pull request #38732: [SPARK-41210][K8S] Window based executor failure tracking mechanism

yaooqinn commented on code in PR #38732: URL: https://github.com/apache/spark/pull/38732#discussion_r1152961738 ## resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodsAllocator.scala: ## @@ -148,6 +163,10 @@ class ExecutorPodsAlloc

[GitHub] [spark] yaooqinn commented on a diff in pull request #38732: [SPARK-41210][K8S] Window based executor failure tracking mechanism

yaooqinn commented on code in PR #38732: URL: https://github.com/apache/spark/pull/38732#discussion_r1153042664 ## resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/Config.scala: ## @@ -750,6 +750,26 @@ private[spark] object Config extends Logging {