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

    https://github.com/apache/spark/pull/21809#discussion_r204633878
  
    --- Diff: core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala ---
    @@ -115,6 +115,8 @@ private[ui] class StagePage(parent: StagesTab, store: 
AppStatusStore) extends We
             return UIUtils.headerSparkPage(request, stageHeader, content, 
parent)
           }
     
    +    val stageJobIds = parent.store.getJobIdsAssociatedWithStage(stageId, 
stageAttemptId)
    --- End diff --
    
    In 
https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala#L109
 there is a query for the stage data already. We can reduce the query to the 
store here.


---

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

Reply via email to