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

    https://github.com/apache/spark/pull/20330#discussion_r162687232
  
    --- Diff: core/src/main/scala/org/apache/spark/ui/jobs/JobPage.scala ---
    @@ -336,8 +336,14 @@ private[ui] class JobPage(parent: JobsTab, store: 
AppStatusStore) extends WebUIP
         content ++= makeTimeline(activeStages ++ completedStages ++ 
failedStages,
           store.executorList(false), appStartTime)
     
    -    content ++= UIUtils.showDagVizForJob(
    -      jobId, store.operationGraphForJob(jobId))
    +    val operationGraphContent = 
store.asOption(store.operationGraphForJob(jobId)) match {
    +      case Some(operationGraph) => UIUtils.showDagVizForJob(jobId, 
operationGraph)
    +      case None =>
    +              <div id="no-info">
    --- End diff --
    
    Indentation is off.


---

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

Reply via email to