[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #38441: [SPARK-40979][CORE] Keep removed executor info due to decommission

2022-11-16 Thread GitBox
dongjoon-hyun commented on code in PR #38441: URL: https://github.com/apache/spark/pull/38441#discussion_r1024561615 ## core/src/test/scala/org/apache/spark/scheduler/TaskSchedulerImplSuite.scala: ## @@ -2017,6 +2019,38 @@ class TaskSchedulerImplSuite extends SparkFunSuite with

[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #38441: [SPARK-40979][CORE] Keep removed executor info due to decommission

2022-11-16 Thread GitBox
dongjoon-hyun commented on code in PR #38441: URL: https://github.com/apache/spark/pull/38441#discussion_r1024560957 ## core/src/test/scala/org/apache/spark/scheduler/TaskSchedulerImplSuite.scala: ## @@ -2017,6 +2019,38 @@ class TaskSchedulerImplSuite extends SparkFunSuite with

[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #38441: [SPARK-40979][CORE] Keep removed executor info due to decommission

2022-11-16 Thread GitBox
dongjoon-hyun commented on code in PR #38441: URL: https://github.com/apache/spark/pull/38441#discussion_r1024247259 ## core/src/test/scala/org/apache/spark/scheduler/TaskSchedulerImplSuite.scala: ## @@ -1976,10 +1977,11 @@ class TaskSchedulerImplSuite extends SparkFunSuite

[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #38441: [SPARK-40979][CORE] Keep removed executor info due to decommission

2022-11-16 Thread GitBox
dongjoon-hyun commented on code in PR #38441: URL: https://github.com/apache/spark/pull/38441#discussion_r1024244358 ## core/src/test/scala/org/apache/spark/scheduler/TaskSchedulerImplSuite.scala: ## @@ -2004,17 +2006,30 @@ class TaskSchedulerImplSuite extends SparkFunSuite

[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #38441: [SPARK-40979][CORE] Keep removed executor info due to decommission

2022-11-16 Thread GitBox
dongjoon-hyun commented on code in PR #38441: URL: https://github.com/apache/spark/pull/38441#discussion_r1024243192 ## core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala: ## @@ -2193,11 +2193,13 @@ private[spark] class DAGScheduler( * Return true when: *

[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #38441: [SPARK-40979][CORE] Keep removed executor info due to decommission

2022-11-16 Thread GitBox
dongjoon-hyun commented on code in PR #38441: URL: https://github.com/apache/spark/pull/38441#discussion_r1024238965 ## core/src/main/scala/org/apache/spark/internal/config/package.scala: ## @@ -2229,6 +2229,16 @@ package object config { .booleanConf

[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #38441: [SPARK-40979][CORE] Keep removed executor info due to decommission

2022-11-16 Thread GitBox
dongjoon-hyun commented on code in PR #38441: URL: https://github.com/apache/spark/pull/38441#discussion_r1024238330 ## core/src/main/scala/org/apache/spark/internal/config/package.scala: ## @@ -2229,6 +2229,16 @@ package object config { .booleanConf