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

    https://github.com/apache/spark/pull/20251#discussion_r161273851
  
    --- 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 lastStage = {
    +      val stageAttempts = jobData.stageIds.flatMap(store.stageData(_))
    +      if (!stageAttempts.isEmpty) {
    +        val lastId = stageAttempts.map(_.stageId).max
    +        stageAttempts.find(_.stageId == lastId)
    --- End diff --
    
    Indeed, nice catch, thx


---

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

Reply via email to