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

    https://github.com/apache/spark/pull/19954#discussion_r158651523
  
    --- Diff: 
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/KubernetesClusterManager.scala
 ---
    @@ -45,6 +45,59 @@ private[spark] class KubernetesClusterManager extends 
ExternalClusterManager wit
           masterURL: String,
           scheduler: TaskScheduler): SchedulerBackend = {
         val sparkConf = sc.getConf
    +    val initContainerConfigMap = 
sparkConf.get(INIT_CONTAINER_CONFIG_MAP_NAME)
    +    val initContainerConfigMapKey = 
sparkConf.get(INIT_CONTAINER_CONFIG_MAP_KEY_CONF)
    +
    +    if (initContainerConfigMap.isEmpty) {
    +      logWarning("The executor's init-container config map was not 
specified. Executors will " +
    +        "therefore not attempt to fetch remote or submitted dependencies.")
    +    }
    +
    +    if (initContainerConfigMapKey.isEmpty) {
    +      logWarning("The executor's init-container config map key was not 
specified. Executors will " +
    --- End diff --
    
    nit: `was not` -> `is not`


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to