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

    https://github.com/apache/spark/pull/18887#discussion_r137942548
  
    --- Diff: 
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
    @@ -229,10 +254,22 @@ private[history] class FsHistoryProvider(conf: 
SparkConf, clock: Clock)
         }
       }
     
    -  override def getListing(): Iterator[FsApplicationHistoryInfo] = 
applications.values.iterator
    +  override def getListing(): Iterator[ApplicationHistoryInfo] = {
    +    listing.view(classOf[ApplicationInfoWrapper])
    +      .index("endTime")
    +      .reverse()
    +      .iterator()
    +      .asScala
    +      .map(_.toAppHistoryInfo)
    --- End diff --
    
    Nit: `toAppHistoryInfo()`


---

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

Reply via email to