Github user onursatici commented on a diff in the pull request:

    https://github.com/apache/spark/pull/23136#discussion_r237534125
  
    --- Diff: 
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodsLifecycleManager.scala
 ---
    @@ -30,7 +30,7 @@ import org.apache.spark.scheduler.ExecutorExited
     import org.apache.spark.util.Utils
     
     private[spark] class ExecutorPodsLifecycleManager(
    -    conf: SparkConf,
    +    val conf: SparkConf,
    --- End diff --
    
    is exposing this fine? maybe we can create a new `eventHandlerUnderTest` in 
the new test below as a workaround


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to