[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/5551 --- 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 enab

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/5551#issuecomment-94115122 This looks fine. The previous setting was very confusing to the user because they would have run into this by following the docs. Thanks for fixing this I'm merging th

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/5551#issuecomment-94039704 Looks fine to me. Seems like it might be useful (and easy) to have different messages for each case, but not a big deal. --- If your project is set up for it, you can rep

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5551#issuecomment-93980637 [Test build #30481 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30481/consoleFull) for PR 5551 at commit [`4335da1`](https://gith

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5551#issuecomment-93980649 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5551#issuecomment-93966845 [Test build #30481 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30481/consoleFull) for PR 5551 at commit [`4335da1`](https://githu

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user srowen commented on the pull request: https://github.com/apache/spark/pull/5551#issuecomment-93965972 Jenkins, retest this please. --- 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: [SPARK-6975][Yarn] Fix argument validation err...

Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5551#issuecomment-93960415 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5551#issuecomment-93960402 [Test build #30471 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30471/consoleFull) for PR 5551 at commit [`4335da1`](https://gith

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5551#issuecomment-93947423 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5551#issuecomment-93947414 [Test build #30466 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30466/consoleFull) for PR 5551 at commit [`77bdcbd`](https://gith

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5551#issuecomment-93944373 [Test build #30471 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30471/consoleFull) for PR 5551 at commit [`4335da1`](https://githu

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user jerryshao commented on a diff in the pull request: https://github.com/apache/spark/pull/5551#discussion_r28577855 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/ClientArguments.scala --- @@ -103,9 +103,14 @@ private[spark] class ClientArguments(args: Array[

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/5551#discussion_r28577520 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/ClientArguments.scala --- @@ -103,9 +103,14 @@ private[spark] class ClientArguments(args: Array[Str

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5551#issuecomment-93928521 [Test build #30466 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30466/consoleFull) for PR 5551 at commit [`77bdcbd`](https://githu

[GitHub] spark pull request: [SPARK-6975][Yarn] Fix argument validation err...

GitHub user jerryshao opened a pull request: https://github.com/apache/spark/pull/5551 [SPARK-6975][Yarn] Fix argument validation error `numExecutors` checking is failed when dynamic allocation is enabled with default configuration. Details can be seen is [SPARK-6975](https://issue