[GitHub] spark pull request: [SPARK-12399] Display correct error message wh...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10352#issuecomment-165378339 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-9578] [ML] Stemmer feature transformer

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

[GitHub] spark pull request: [SPARK-12399] Display correct error message wh...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10352#issuecomment-165378338 Merged build finished. Test FAILed. --- 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: [SPARK-12218] [SQL] [Backport-1.5] Fixed the P...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10344#issuecomment-165378986 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: [SPARK-11627] Add initial input rate limit for...

2015-12-17 Thread BenFradet
Github user BenFradet commented on a diff in the pull request: https://github.com/apache/spark/pull/9593#discussion_r47878665 --- Diff: docs/configuration.md --- @@ -1523,6 +1523,15 @@ Apart from these, the following properties are also available, and may be useful

[GitHub] spark pull request: [SPARK-12218] [SQL] [Backport-1.5] Fixed the P...

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

[GitHub] spark pull request: [SPARK-11627] Add initial input rate limit for...

2015-12-17 Thread BenFradet
Github user BenFradet commented on a diff in the pull request: https://github.com/apache/spark/pull/9593#discussion_r47878960 --- Diff: docs/configuration.md --- @@ -1523,6 +1523,15 @@ Apart from these, the following properties are also available, and may be useful

[GitHub] spark pull request: [SPARK-12395] [SQL] fix resulting columns of o...

2015-12-17 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/10353#issuecomment-165380331 cc @rxin @liancheng --- 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

[GitHub] spark pull request: [SPARK-12395] [SQL] fix resulting columns of o...

2015-12-17 Thread davies
GitHub user davies opened a pull request: https://github.com/apache/spark/pull/10353 [SPARK-12395] [SQL] fix resulting columns of outer join For API DataFrame.join(right, usingColumns, joinType), if the joinType is right_outer or full_outer, the resulting join columns could be

[GitHub] spark pull request: [SPARK-11627] Add initial input rate limit for...

2015-12-17 Thread BenFradet
Github user BenFradet commented on a diff in the pull request: https://github.com/apache/spark/pull/9593#discussion_r47879107 --- Diff: docs/configuration.md --- @@ -1523,6 +1523,15 @@ Apart from these, the following properties are also available, and may be useful

[GitHub] spark pull request: [SPARK-11627] Add initial input rate limit for...

2015-12-17 Thread BenFradet
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/9593#issuecomment-165380452 A few remarks regarding the doc. --- 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-12388] change default compression to lz...

2015-12-17 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/10342#issuecomment-165380956 I had patched LZ4BlockInputStream to support concated streams. --- 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-12399] Display correct error message wh...

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

[GitHub] spark pull request: [SPARK-12388] change default compression to lz...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10342#issuecomment-165389625 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-8519] [ML] [MLlib] Blockify distance co...

2015-12-17 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/10306#discussion_r47878122 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/clustering/KMeans.scala --- @@ -250,114 +240,142 @@ class KMeans private ( }

[GitHub] spark pull request: [SPARK-12218] [SQL] [Backport-1.5] Fixed the P...

2015-12-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10344#issuecomment-165378831 [Test build #47899 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47899/console) for PR 10344 at commit

[GitHub] spark pull request: [SPARK-9578] [ML] Stemmer feature transformer

2015-12-17 Thread BenFradet
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/10272#issuecomment-165378793 LGTM --- 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

[GitHub] spark pull request: [SPARK-12149] [Web UI] Executor UI improvement...

2015-12-17 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10154#issuecomment-165378836 Jenkins test 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-12149] [Web UI] Executor UI improvement...

2015-12-17 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10154#issuecomment-165378870 Jenkins add to whitelist --- 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-12218] [SQL] Fixed the Parquet's filter...

2015-12-17 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/10278#issuecomment-165379694 @gatorsmile Sorry for the late reply and thanks for the nice catch! The `In` predicate push down issue had been tracked by SPARK-11164, and done as part of

[GitHub] spark pull request: [SPARK-11627] Add initial input rate limit for...

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

[GitHub] spark pull request: Once driver register successfully, stop it to ...

2015-12-17 Thread echoTomei
GitHub user echoTomei opened a pull request: https://github.com/apache/spark/pull/10354 Once driver register successfully, stop it to connect to master. This commit is to resolve SPARK-12396. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] spark pull request: [SPARK-8519] [ML] [MLlib] Blockify distance co...

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

[GitHub] spark pull request: [SPARK-8519] [ML] [MLlib] Blockify distance co...

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

[GitHub] spark pull request: [SPARK-12397][SQL] Improve error messages for ...

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

[GitHub] spark pull request: [SPARK-12397][SQL] Improve error messages for ...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10351#issuecomment-165388269 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: [SPARK-12388] change default compression to lz...

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

[GitHub] spark pull request: [SPARK-12388] change default compression to lz...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10342#issuecomment-165389624 Merged build finished. Test FAILed. --- 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: [SPARK-7425] [ML] [WIP] spark.ml Predictor sho...

2015-12-17 Thread BenFradet
GitHub user BenFradet opened a pull request: https://github.com/apache/spark/pull/10355 [SPARK-7425] [ML] [WIP] spark.ml Predictor should support other numeric types for label Currently, the Predictor abstraction expects the input labelCol type to be DoubleType, but we should

[GitHub] spark pull request: [SPARK-7425] [ML] [WIP] spark.ml Predictor sho...

2015-12-17 Thread BenFradet
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/10355#issuecomment-165389989 This is still a wip but remarks are welcome. --- 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: [SPARK-12289][SQL] Support UnsafeRow in TakeOr...

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

[GitHub] spark pull request: [SPARK-11215] [ML] Add multiple columns suppor...

2015-12-17 Thread BenFradet
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/9183#issuecomment-165387557 +1, I've been meaning to request this transformer for a while. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-9578] [ML] Stemmer feature transformer

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10272#issuecomment-165378488 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: [SPARK-8519] [ML] [MLlib] Blockify distance co...

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

[GitHub] spark pull request: [SPARK-9578] [ML] Stemmer feature transformer

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

[GitHub] spark pull request: [SPARK-12368] [ML] [DOC] Better doc for the bi...

2015-12-17 Thread BenFradet
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/10328#issuecomment-165380597 cc @jkbradley @mengxr --- 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-12054] [SQL] Consider nullability of ex...

2015-12-17 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/10333#discussion_r47879128 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameJoinSuite.scala --- @@ -48,11 +48,13 @@ class DataFrameJoinSuite extends QueryTest with

[GitHub] spark pull request: [SPARK-11627] Add initial input rate limit for...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9593#issuecomment-165381420 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: [SPARK-12149] [Web UI] Executor UI improvement...

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

[GitHub] spark pull request: [SPARK-12395] [SQL] fix resulting columns of o...

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

[GitHub] spark pull request: [SPARK-11627] Add initial input rate limit for...

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

[GitHub] spark pull request: Once driver register successfully, stop it to ...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10354#issuecomment-165385604 Can one of the admins verify this patch? --- 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: [SPARK-8519] [ML] [MLlib] Blockify distance co...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10306#issuecomment-165385372 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: [SPARK-12397][SQL] Improve error messages for ...

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

[GitHub] spark pull request: [SPARK-7425] [ML] [WIP] spark.ml Predictor sho...

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

[GitHub] spark pull request: [SPARK-7425] [ML] [WIP] spark.ml Predictor sho...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10355#issuecomment-165390848 Merged build finished. Test FAILed. --- 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: [SPARK-7425] [ML] [WIP] spark.ml Predictor sho...

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

[GitHub] spark pull request: [SPARK-12293][SQL] Support UnsafeRow in LocalT...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10283#issuecomment-165393360 Merged build finished. Test FAILed. --- 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: [SPARK-12293][SQL] Support UnsafeRow in LocalT...

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

[GitHub] spark pull request: [SPARK-12293][SQL] Support UnsafeRow in LocalT...

2015-12-17 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10283#issuecomment-165396392 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 this

[GitHub] spark pull request: [SPARK-11627] Add initial input rate limit for...

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

[GitHub] spark pull request: [SPARK-11627] Add initial input rate limit for...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9593#issuecomment-165399693 Merged build finished. Test FAILed. --- 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: [SPARK-11627] Add initial input rate limit for...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9593#issuecomment-165399695 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12377][Python][Wrong implementation for...

2015-12-17 Thread somideshmukh
GitHub user somideshmukh opened a pull request: https://github.com/apache/spark/pull/10356 [SPARK-12377][Python][Wrong implementation for Row.__call__ in pyspark] Made chnages in types.py file to change "return _create_row(self, args)" to return create_row(self.fields_, args) You

[GitHub] spark pull request: [SPARK-12377][Python][Wrong implementation for...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10356#issuecomment-165404400 Can one of the admins verify this patch? --- 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: [SPARK-12289][SQL] Support UnsafeRow in TakeOr...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10330#issuecomment-165405876 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: [SPARK-12404] [SQL] Ensure objects passed to S...

2015-12-17 Thread sarutak
GitHub user sarutak opened a pull request: https://github.com/apache/spark/pull/10357 [SPARK-12404] [SQL] Ensure objects passed to StaticInvoke is Serializable Now `StaticInvoke` receives Any as a object and StaticInvoke can be serialized but sometimes the object passed is not

[GitHub] spark pull request: [SPARK-12289][SQL] Support UnsafeRow in TakeOr...

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

[GitHub] spark pull request: [SPARK-12404] [SQL] Ensure objects passed to S...

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

[GitHub] spark pull request: [SPARK-12149] [Web UI] Executor UI improvement...

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

[GitHub] spark pull request: [SPARK-12388] change default compression to lz...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10342#issuecomment-165391692 Merged build finished. Test FAILed. --- 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: [SPARK-12388] change default compression to lz...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10342#issuecomment-165391694 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12388] change default compression to lz...

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

[GitHub] spark pull request: [SPARK-12293][SQL] Support UnsafeRow in LocalT...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10283#issuecomment-165393366 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-7425] [ML] [WIP] spark.ml Predictor sho...

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

[GitHub] spark pull request: [SPARK-12289][SQL] Support UnsafeRow in TakeOr...

2015-12-17 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10330#discussion_r47891294 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala --- @@ -207,7 +211,7 @@ case class TakeOrderedAndProject(

[GitHub] spark pull request: [SPARK-7425] [ML] [WIP] spark.ml Predictor sho...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10355#issuecomment-165390849 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-7425] [ML] [WIP] spark.ml Predictor sho...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10355#issuecomment-165396126 Merged build finished. Test FAILed. --- 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: [SPARK-7425] [ML] [WIP] spark.ml Predictor sho...

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

[GitHub] spark pull request: [SPARK-7425] [ML] [WIP] spark.ml Predictor sho...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10355#issuecomment-165396129 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12293][SQL] Support UnsafeRow in LocalT...

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

[GitHub] spark pull request: [SPARK-7425] [ML] [WIP] spark.ml Predictor sho...

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

[GitHub] spark pull request: [SPARK-7425] [ML] [WIP] spark.ml Predictor sho...

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

[GitHub] spark pull request: [SPARK-12289][SQL] Support UnsafeRow in TakeOr...

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

[GitHub] spark pull request: [SPARK-12149] [Web UI] Executor UI improvement...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10154#issuecomment-165408794 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: [SPARK-12149] [Web UI] Executor UI improvement...

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

[GitHub] spark pull request: [SPARK-12404] [SQL] Ensure objects passed to S...

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

[GitHub] spark pull request: [SPARK-12345] [CORE] Do not send SPARK_HOME th...

2015-12-17 Thread skyluc
Github user skyluc commented on the pull request: https://github.com/apache/spark/pull/10329#issuecomment-165459198 Fixed the 'style', added a comment, and switch to `filterKeys`. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-12366][SQL][REPL] ExecutorClassLoader s...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10358#issuecomment-165454564 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12366][SQL][REPL] ExecutorClassLoader s...

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

[GitHub] spark pull request: [SPARK-12345][MESOS] Filter SPARK_HOME when su...

2015-12-17 Thread dragos
Github user dragos commented on a diff in the pull request: https://github.com/apache/spark/pull/10332#discussion_r47902198 --- Diff: core/src/main/scala/org/apache/spark/deploy/rest/mesos/MesosRestServer.scala --- @@ -94,7 +94,12 @@ private[mesos] class MesosSubmitRequestServlet(

[GitHub] spark pull request: [SPARK-9384] [core] Easier setting of executor...

2015-12-17 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/7739#issuecomment-165466946 I'm a bit on the fence about this. Its kind of bringing us full circle to what SPARK used to have when they had env variables, most of those applied across

[GitHub] spark pull request: [SPARK-12404] [SQL] Ensure objects passed to S...

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

[GitHub] spark pull request: [SPARK-12404] [SQL] Ensure objects passed to S...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10357#issuecomment-165451412 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: [SPARK-12366][SQL][REPL] ExecutorClassLoader s...

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

[GitHub] spark pull request: [SPARK-12345] [CORE] Do not send SPARK_HOME th...

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

[GitHub] spark pull request: [SPARK-12345][MESOS] Filter SPARK_HOME when su...

2015-12-17 Thread jerryshao
Github user jerryshao commented on a diff in the pull request: https://github.com/apache/spark/pull/10332#discussion_r47909261 --- Diff: core/src/main/scala/org/apache/spark/deploy/rest/mesos/MesosRestServer.scala --- @@ -94,7 +94,12 @@ private[mesos] class

[GitHub] spark pull request: [SPARK-12366][SQL][REPL] ExecutorClassLoader s...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10358#issuecomment-165454563 Merged build finished. Test FAILed. --- 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: [SPARK-12404] [SQL] Ensure objects passed to S...

2015-12-17 Thread yu-iskw
Github user yu-iskw commented on the pull request: https://github.com/apache/spark/pull/10357#issuecomment-165461189 @sarutak thank you for sending this PR. @marmbrus @rxin could you review it? I think this is a little big issue. We should fix it before releasing Spark 1.6.

[GitHub] spark pull request: [SPARK-11749][Streaming] Duplicate creating th...

2015-12-17 Thread jhu-chang
Github user jhu-chang commented on the pull request: https://github.com/apache/spark/pull/9765#issuecomment-165470678 @zsxwing Thanks for your comments, could you help to check this again? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-12366][SQL][REPL] ExecutorClassLoader s...

2015-12-17 Thread sarutak
Github user sarutak commented on the pull request: https://github.com/apache/spark/pull/10358#issuecomment-165455309 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 this

[GitHub] spark pull request: [SPARK-12376][TESTS] Spark Streaming Java8APIS...

2015-12-17 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10336#issuecomment-165480439 LGTM --- 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

[GitHub] spark pull request: [SPARK-12321][SQL] JSON format for TreeNode (u...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10311#issuecomment-165492038 Merged build finished. Test FAILed. --- 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: Once driver register successfully, stop it to ...

2015-12-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10354 --- 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

[GitHub] spark pull request: Once driver register successfully, stop it to ...

2015-12-17 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/10354#issuecomment-165494701 This PR was merged by mistake, reverted, sorry. --- 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: [SPARK-12054] [SQL] Consider nullability of ex...

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

[GitHub] spark pull request: [SPARK-12345][Mesos] Properly filter out SPARK...

2015-12-17 Thread dragos
Github user dragos commented on a diff in the pull request: https://github.com/apache/spark/pull/10359#discussion_r47929981 --- Diff: core/src/main/scala/org/apache/spark/deploy/rest/mesos/MesosRestServer.scala --- @@ -99,7 +99,7 @@ private[mesos] class MesosSubmitRequestServlet(

[GitHub] spark pull request: [SPARK-12366][SQL][REPL] ExecutorClassLoader s...

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

[GitHub] spark pull request: [SPARK-12366][SQL][REPL] ExecutorClassLoader s...

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

[GitHub] spark pull request: [SPARK-12311][CORE] Restore previous value of ...

2015-12-17 Thread kiszk
Github user kiszk commented on a diff in the pull request: https://github.com/apache/spark/pull/10289#discussion_r47920442 --- Diff: core/src/test/scala/org/apache/spark/util/collection/ExternalSorterSuite.scala --- @@ -235,7 +235,7 @@ class ExternalSorterSuite extends

  1   2   3   4   5   6   >