[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45457729 Merged build triggered. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45458843 Merged build started. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45460079 Merged build finished. All automated tests passed. --- 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: [WIP][SPARK-1930] The Container is running bey...

2014-06-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45460081 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15553/ --- If your project

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-06 Thread mridulm
Github user mridulm commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45317843 I meant, I was expecting the default 3xx mb to be overhead : while this ended up making it 2.5gig or so : causing master to fail due to inadequate memory to run the job.

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45319395 Merged build triggered. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45319768 Merged build triggered. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45333612 Merged build started. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45337362 Merged build finished. All automated tests passed. --- 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: [WIP][SPARK-1930] The Container is running bey...

2014-06-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45337364 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15506/ --- If your project

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45337367 Merged build started. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45341590 Merged build finished. All automated tests passed. --- 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: [WIP][SPARK-1930] The Container is running bey...

2014-06-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45341591 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15507/ --- If your project

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-05 Thread mridulm
Github user mridulm commented on a diff in the pull request: https://github.com/apache/spark/pull/894#discussion_r13471878 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala --- @@ -65,6 +65,18 @@ trait ClientBase extends Logging { val

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-05 Thread witgo
Github user witgo commented on a diff in the pull request: https://github.com/apache/spark/pull/894#discussion_r13473687 --- Diff: yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala --- @@ -65,6 +65,18 @@ trait ClientBase extends Logging { val

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45055403 Merged build finished. All automated tests passed. --- 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: [WIP][SPARK-1930] The Container is running bey...

2014-06-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45055404 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15434/ --- If your project

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45053440 Merged build triggered. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-45053449 Merged build started. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-06-01 Thread witgo
Github user witgo commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-4410 @mridulm The following code in line with your thoughts? https://github.com/witgo/spark/compare/SPARK-1930_different --- If your project is set up for it, you can

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-31 Thread witgo
Github user witgo commented on a diff in the pull request: https://github.com/apache/spark/pull/894#discussion_r13259895 --- Diff: yarn/alpha/src/main/scala/org/apache/spark/deploy/yarn/ExecutorLauncher.scala --- @@ -92,21 +92,22 @@ class ExecutorLauncher(args:

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44730423 Merged build started. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44730282 Merged build triggered. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44742298 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15316/ --- If your project

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44742297 Merged build finished. All automated tests passed. --- 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: [WIP][SPARK-1930] The Container is running bey...

2014-05-30 Thread mridulm
Github user mridulm commented on a diff in the pull request: https://github.com/apache/spark/pull/894#discussion_r13220194 --- Diff: yarn/alpha/src/main/scala/org/apache/spark/deploy/yarn/ExecutorLauncher.scala --- @@ -92,21 +92,22 @@ class ExecutorLauncher(args:

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-30 Thread mridulm
Github user mridulm commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44625937 hey @sryza did we also add overhead for workers too ? iirc I had added it only for master. Do we need two different values for both ? Since the performance

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-30 Thread sryza
Github user sryza commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44666824 Workers currently get 384 added as well (see YarnAllocationHandler.MEMORY_OVERHEARD). At a first approximation, this seems reasonable to me. The AM and executors are both

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44369782 Merged build started. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44369775 Merged build triggered. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44372220 Merged build finished. All automated tests passed. --- 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: [WIP][SPARK-1930] The Container is running bey...

2014-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44372221 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15246/ --- If your project

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44407643 Merged build triggered. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44407668 Merged build started. --- 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

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-28 Thread witgo
Github user witgo commented on a diff in the pull request: https://github.com/apache/spark/pull/894#discussion_r13131123 --- Diff: yarn/stable/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocationHandler.scala --- @@ -90,6 +90,12 @@ private[yarn] class YarnAllocationHandler(

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44415303 Merged build finished. All automated tests passed. --- 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: [WIP][SPARK-1930] The Container is running bey...

2014-05-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44415304 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15252/ --- If your project

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-28 Thread witgo
Github user witgo commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44424182 I agree with @sryza .Spark automatically handle these better. Of course, we can allow users to manually specify the special value. --- If your project is set up for it,

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44364586 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15242/ --- If your project

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-27 Thread mridulm
Github user mridulm commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44366613 The entire process tree is tracked ... Note that yarn allocates in multiples of memory slots and kills only when the container requirement is violated. On

[GitHub] spark pull request: [WIP][SPARK-1930] The Container is running bey...

2014-05-27 Thread sryza
Github user sryza commented on the pull request: https://github.com/apache/spark/pull/894#issuecomment-44366899 Agree with @tgravescs and @mridulm that a constant overhead makes more sense. @pwendell YARN includes the memory usage of subprocesses in its calculation.