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

    https://github.com/apache/spark/pull/19920#discussion_r155603886
  
    --- Diff: core/src/main/scala/org/apache/spark/status/api/v1/api.scala ---
    @@ -24,27 +24,32 @@ import 
com.fasterxml.jackson.databind.annotation.JsonDeserialize
     
     import org.apache.spark.JobExecutionStatus
     
    -class ApplicationInfo private[spark](
    -    val id: String,
    -    val name: String,
    -    val coresGranted: Option[Int],
    -    val maxCores: Option[Int],
    -    val coresPerExecutor: Option[Int],
    -    val memoryPerExecutorMB: Option[Int],
    -    val attempts: Seq[ApplicationAttemptInfo])
    +case class ApplicationInfo private[spark](
    --- End diff --
    
    What is turning this into a case class achieving? I only see noise caused 
by this change.


---

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

Reply via email to