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

    https://github.com/apache/spark/pull/22215#discussion_r214612510
  
    --- Diff: 
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/KubernetesUtils.scala
 ---
    @@ -60,4 +64,81 @@ private[spark] object KubernetesUtils {
       }
     
       def parseMasterUrl(url: String): String = url.substring("k8s://".length)
    +
    +  def formatPairsBundle(pairs: Seq[(String, String)], indent: Int = 1) : 
String = {
    +    // Use more loggable format if value is null or empty
    +    val indentStr = "\t" * indent
    --- End diff --
    
    I just preserved the original codes choice here, I would happily change to 
spaces if preferred


---

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

Reply via email to