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

    https://github.com/apache/spark/pull/19920#discussion_r155657306
  
    --- Diff: 
core/src/main/scala/org/apache/spark/deploy/history/HistoryPage.scala ---
    @@ -88,4 +90,9 @@ private[history] class HistoryPage(parent: HistoryServer) 
extends WebUIPage("")
       private def makePageLink(showIncomplete: Boolean): String = {
         UIUtils.prependBaseUri("/?" + "showIncomplete=" + showIncomplete)
       }
    +
    +  private def isApplicationCompleted(appInfo: ApplicationInfo): Boolean = {
    +    appInfo.attempts.nonEmpty && appInfo.attempts.head.completed
    +
    --- End diff --
    
    nuked


---

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

Reply via email to