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

    https://github.com/apache/spark/pull/20251#discussion_r161293689
  
    --- Diff: core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala ---
    @@ -429,20 +429,40 @@ private[ui] class JobDataSource(
         val formattedDuration = duration.map(d => 
UIUtils.formatDuration(d)).getOrElse("Unknown")
         val submissionTime = jobData.submissionTime
         val formattedSubmissionTime = 
submissionTime.map(UIUtils.formatDate).getOrElse("Unknown")
    -    val jobDescription = 
UIUtils.makeDescription(jobData.description.getOrElse(""),
    -      basePath, plainText = false)
    +
    +    val lastStageAttempt = {
    +      val stageAttempts = jobData.stageIds.flatMap(store.stageData(_))
    --- End diff --
    
    Do we want to have data of the last stage or the last stageAttempt? 
    In the second case we should load only the last stage, indeed.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to