Github user dongjoon-hyun commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22577#discussion_r221410603
  
    --- Diff: 
core/src/main/scala/org/apache/spark/status/api/v1/OneApplicationResource.scala 
---
    @@ -175,7 +175,7 @@ private[v1] class OneApplicationAttemptResource extends 
AbstractApplicationResou
       def getAttempt(): ApplicationAttemptInfo = {
         uiRoot.getApplicationInfo(appId)
           .flatMap { app =>
    -        app.attempts.filter(_.attemptId == attemptId).headOption
    --- End diff --
    
    @sadhen Since this is a general code fix for 
`master/branch-2.4/branch-2.3`. Can we update the title without mentioning 
`compiling for Scala 2.12.7`?
    > branch-2.2 is fine, but branch-2.3 should be fixed.


---

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

Reply via email to