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

    https://github.com/apache/spark/pull/21748#discussion_r202753060
  
    --- Diff: 
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/KubernetesClusterManager.scala
 ---
    @@ -35,26 +35,39 @@ private[spark] class KubernetesClusterManager extends 
ExternalClusterManager wit
       override def canCreate(masterURL: String): Boolean = 
masterURL.startsWith("k8s")
     
       override def createTaskScheduler(sc: SparkContext, masterURL: String): 
TaskScheduler = {
    -    if (masterURL.startsWith("k8s") &&
    -      sc.deployMode == "client" &&
    -      !sc.conf.get(KUBERNETES_DRIVER_SUBMIT_CHECK).getOrElse(false)) {
    -      throw new SparkException("Client mode is currently not supported for 
Kubernetes.")
    -    }
    -
         new TaskSchedulerImpl(sc)
       }
     
       override def createSchedulerBackend(
           sc: SparkContext,
           masterURL: String,
           scheduler: TaskScheduler): SchedulerBackend = {
    +    val wasSparkSubmittedInClusterMode = 
sc.conf.get(KUBERNETES_DRIVER_SUBMIT_CHECK)
    +    val (authConfPrefix,
    +      apiServerUri,
    +      defaultServiceAccountToken,
    +      defaultServiceAccountCaCrt) = if (wasSparkSubmittedInClusterMode) {
    +      require(sc.conf.get(KUBERNETES_DRIVER_POD_NAME).isDefined,
    +        "If the application is deployed using spark-submit in cluster 
mode, the driver pod name" +
    +          " must be provided.")
    --- End diff --
    
    Empty space should be after the previous line.


---

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

Reply via email to