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

    https://github.com/apache/spark/pull/20481#discussion_r165544206
  
    --- Diff: core/src/main/scala/org/apache/spark/status/storeTypes.scala ---
    @@ -73,6 +73,8 @@ private[spark] class JobDataWrapper(
       @JsonIgnore @KVIndex
       private def id: Int = info.jobId
     
    +  @JsonIgnore @KVIndex("completionTime")
    +  private def completionTime: Long = 
info.completionTime.map(_.getTime).getOrElse(-1)
    --- End diff --
    
    nit: `-1` -> `-1L`


---

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

Reply via email to