[GitHub] spark pull request #18707: [SPARK-21503][UI]: Fixed the issue

[pgandhi999]

GitHub user pgandhi999 opened a pull request:

    https://github.com/apache/spark/pull/18707

    [SPARK-21503][UI]: Fixed the issue

    Added the case ExecutorLostFailure which was previously not there, thus, 
the default case would be executed in which case, task would be marked as 
completed.
    
    ## What changes were proposed in this pull request?
          Added the case ExecutorLostFailure in the ExecutorsTab.scala class, 
which will consider all those cases where executor connection to Spark Driver 
was lost due to killing the executor process, network connection etc.
    
    
    
    ## How was this patch tested?
    Manually Tested the fix by observing the UI change before and after.
    Before:
    <img width="1398" alt="screen shot-before" 
src="https://user-images.githubusercontent.com/22228190/28482929-571c9cea-6e30-11e7-93dd-728de5cdea95.png";>
    After:
    <img width="1385" alt="screen shot-after" 
src="https://user-images.githubusercontent.com/22228190/28482964-8649f5ee-6e30-11e7-91bd-2eb2089c61cc.png";>
    
    
    Please review http://spark.apache.org/contributing.html before opening a 
pull request.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/pgandhi999/spark master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/18707.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #18707
    
----
commit 172fc20898896058b7288360eb5292ed9df9d79c
Author: pgandhi <pgan...@yahoo-inc.com>
Date:   2017-07-21T21:00:22Z

    [SPARK-21503]: Fixed the issue
    
    Added the case ExecutorLostFailure which was previously not there, thus, 
the default case would be executed in which case, task would be marked as 
completed.

----


---
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