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

    https://github.com/apache/spark/pull/19399#discussion_r143591603
  
    --- Diff: 
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
    @@ -833,10 +850,12 @@ private[history] class AppListingListener(log: 
FileStatus, clock: Clock) extends
         var sparkUser: String = null
         var completed = false
         var appSparkVersion = ""
    +    val jobToStatus = new mutable.HashMap[Int, String]
    --- End diff --
    
    I suggest that the value type can be Boolean instead of String, or use a 
HashSet to mark either the successful ones or failed ones.


---

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

Reply via email to