Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20330#discussion_r162762976
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala ---
@@ -1002,4 +1000,12 @@ private object ApiHelper {
}
}
+ def lastStageNameAndDescription(store: AppStatusStore, job: JobData):
(String, String) = {
+ store.asOption(store.lastStageAttempt(job.stageIds.max)) match {
+ case Some(lastStageAttempt) =>
+ (lastStageAttempt.name,
lastStageAttempt.description.getOrElse(job.name))
+ case None => ("", "")
--- End diff --
Before, you were doing `if (lastStageDescription.isEmpty) job.name else
blah` at the call site.
Now, when the last stage is not in the store, the call site is getting an
empty string as the description, instead of using the job name.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
