[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-20 Thread archit279thakur
Github user archit279thakur commented on the pull request: https://github.com/apache/spark/pull/6263#issuecomment-158366439 @squito In reply to: *@archit279thakur can you also bring this up to date with master, and include before & after screenshots? I'd like for thi

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-16 Thread archit279thakur
Github user archit279thakur commented on a diff in the pull request: https://github.com/apache/spark/pull/6263#discussion_r45029094 --- Diff: core/src/main/scala/org/apache/spark/storage/StorageStatusListener.scala --- @@ -87,6 +113,8 @@ class StorageStatusListener extends

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-15 Thread archit279thakur
Github user archit279thakur commented on the pull request: https://github.com/apache/spark/pull/6263#issuecomment-156944200 It says: Error: Invalid or corrupt jarfile build/sbt-launch-0.13.7.jar Should I bring it up to date with master first? --- If your project is set up for it

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-15 Thread archit279thakur
Github user archit279thakur commented on the pull request: https://github.com/apache/spark/pull/6263#issuecomment-156940368 jenkins, please test again. --- 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

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-15 Thread archit279thakur
Github user archit279thakur commented on a diff in the pull request: https://github.com/apache/spark/pull/6263#discussion_r44892842 --- Diff: core/src/main/scala/org/apache/spark/storage/StorageStatusListener.scala --- @@ -28,15 +36,33 @@ import org.apache.spark.scheduler

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-05 Thread archit279thakur
Github user archit279thakur commented on a diff in the pull request: https://github.com/apache/spark/pull/6263#discussion_r44062329 --- Diff: core/src/main/scala/org/apache/spark/storage/StorageStatusListener.scala --- @@ -17,26 +17,55 @@ package

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-05 Thread archit279thakur
Github user archit279thakur commented on a diff in the pull request: https://github.com/apache/spark/pull/6263#discussion_r44059357 --- Diff: core/src/main/scala/org/apache/spark/storage/StorageStatusListener.scala --- @@ -17,26 +17,55 @@ package

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-05 Thread archit279thakur
Github user archit279thakur commented on a diff in the pull request: https://github.com/apache/spark/pull/6263#discussion_r44059314 --- Diff: core/src/main/scala/org/apache/spark/storage/StorageStatusListener.scala --- @@ -17,26 +17,55 @@ package

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-05 Thread archit279thakur
Github user archit279thakur commented on a diff in the pull request: https://github.com/apache/spark/pull/6263#discussion_r44059300 --- Diff: core/src/test/scala/org/apache/spark/storage/StorageStatusListenerSuite.scala --- @@ -17,10 +17,16 @@ package

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-05 Thread archit279thakur
Github user archit279thakur commented on the pull request: https://github.com/apache/spark/pull/6263#issuecomment-154167926 @squito Thanks for your comments. Incorporated them all and also gone through the link. Please point out if I missed anything. --- If your project is set

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-05 Thread archit279thakur
Github user archit279thakur commented on a diff in the pull request: https://github.com/apache/spark/pull/6263#discussion_r44056583 --- Diff: core/src/main/scala/org/apache/spark/storage/StorageStatusListener.scala --- @@ -17,26 +17,55 @@ package

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-05 Thread archit279thakur
Github user archit279thakur commented on a diff in the pull request: https://github.com/apache/spark/pull/6263#discussion_r44056239 --- Diff: core/src/main/scala/org/apache/spark/storage/StorageStatusListener.scala --- @@ -17,26 +17,55 @@ package

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-05 Thread archit279thakur
Github user archit279thakur commented on the pull request: https://github.com/apache/spark/pull/6263#issuecomment-154155643 @squito Can you please review the last two commits. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-04 Thread archit279thakur
Github user archit279thakur commented on a diff in the pull request: https://github.com/apache/spark/pull/6263#discussion_r43982245 --- Diff: core/src/test/scala/org/apache/spark/storage/StorageStatusListenerSuite.scala --- @@ -150,4 +158,12 @@ class StorageStatusListenerSuite

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-11-04 Thread archit279thakur
Github user archit279thakur commented on the pull request: https://github.com/apache/spark/pull/6263#issuecomment-153674563 @suyanNone Can you please review my 2nd commit. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-10-27 Thread archit279thakur
Github user archit279thakur commented on the pull request: https://github.com/apache/spark/pull/6263#issuecomment-151574473 Sure, and time for expiration should be configuration based? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: SPARK-7729:Executor which has been killed shou...

2015-05-19 Thread archit279thakur
GitHub user archit279thakur opened a pull request: https://github.com/apache/spark/pull/6263 SPARK-7729:Executor which has been killed should also be displayed on… … Executors Tab. You can merge this pull request into a Git repository by running: $ git pull https