[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150589550 **[Test build #44223 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44223/consoleFull)** for PR 9232 at commit

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread steveloughran
Github user steveloughran commented on a diff in the pull request: https://github.com/apache/spark/pull/9232#discussion_r42866102 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala --- @@ -1337,55 +1337,8 @@ object Client extends Logging { conf:

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread steveloughran
Github user steveloughran commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150579863 -just pushed up a version with @dougb's suggestion; the check for cluster mode is made before trying to obtain either hive or hbase tokens. Note that the

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150579870 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

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150577860 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

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150579433 **[Test build #44222 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44222/consoleFull)** for PR 9232 at commit

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150581810 **[Test build #44223 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44223/consoleFull)** for PR 9232 at commit

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150586249 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150577826 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: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread steveloughran
Github user steveloughran commented on a diff in the pull request: https://github.com/apache/spark/pull/9232#discussion_r42866295 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala --- @@ -337,5 +339,82 @@ object YarnSparkHadoopUtil {

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150579839 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: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150589707 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150586251 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150589711 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-22 Thread steveloughran
GitHub user steveloughran opened a pull request: https://github.com/apache/spark/pull/9232 Stevel/patches/spark 11265 hive tokens This is just the first "it compiles" stage of the patch: it doesn't contain any fixes I've just pulled the current code out of the yarn client

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150360112 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

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150366139 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150366002 **[Test build #44176 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44176/consoleFull)** for PR 9232 at commit

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150366133 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-22 Thread chesterxgchen
Github user chesterxgchen commented on a diff in the pull request: https://github.com/apache/spark/pull/9232#discussion_r42810729 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala --- @@ -337,5 +339,82 @@ object YarnSparkHadoopUtil {

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-22 Thread chesterxgchen
Github user chesterxgchen commented on a diff in the pull request: https://github.com/apache/spark/pull/9232#discussion_r42810773 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala --- @@ -337,5 +339,82 @@ object YarnSparkHadoopUtil {

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150361708 **[Test build #44176 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44176/consoleFull)** for PR 9232 at commit

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-22 Thread dougb
Github user dougb commented on a diff in the pull request: https://github.com/apache/spark/pull/9232#discussion_r42809360 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala --- @@ -1337,55 +1337,8 @@ object Client extends Logging { conf:

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-22 Thread chesterxgchen
Github user chesterxgchen commented on a diff in the pull request: https://github.com/apache/spark/pull/9232#discussion_r42810639 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala --- @@ -337,5 +339,82 @@ object YarnSparkHadoopUtil {

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-22 Thread chesterxgchen
Github user chesterxgchen commented on a diff in the pull request: https://github.com/apache/spark/pull/9232#discussion_r42811237 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala --- @@ -337,5 +339,82 @@ object YarnSparkHadoopUtil {

[GitHub] spark pull request: Stevel/patches/spark 11265 hive tokens

2015-10-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9232#issuecomment-150360049 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