[GitHub] spark pull request #22215: [SPARK-25222][K8S] Improve container status loggi...

[rvesse <gi...@git.apache.org>]

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: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org