Github user squito commented on the pull request:

    https://github.com/apache/spark/pull/6263#issuecomment-159409155
  
    Hi @archit279thakur,
    
    good questions about what to do with the time it was killed.   The reason I 
wanted it included is so the user could put it together with the timeline, to 
see what stages were running when the executor was killed, to help them debug 
*why* the executor was removed.  I dont' have strong opinions about where it 
should go in the UI -- I'm willing to believe that it will just lead to too 
much clutter.  @CodingCat , any thoughts?  But in either case, it would still 
be nice to have in the json endpoint.
    
    btw, the mima failure is from this:
    
    ```
    [error]  * method 
this(java.lang.String,java.lang.String,Int,Long,Long,Int,Int,Int,Int,Long,Long,Long,Long,Long,scala.collection.Map)Unit
 in class org.apache.spark.status.api.v1.ExecutorSummary does not have a 
correspondent in new version
    [error]    filter with: 
ProblemFilters.exclude[MissingMethodProblem]("org.apache.spark.status.api.v1.ExecutorSummary.this")
    ```
    you can add that line to `project/MimaExcludes.scala`, that constructor is 
private so this a false positive.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to