[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108064998 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: [SPARK-7529] [mllib] Added several Java-friend...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6562#issuecomment-108065016 [Test build #34005 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34005/consoleFull) for PR 6562 at commit [`c16821b`](https://git

[GitHub] spark pull request: just for testing ...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108065053 [Test build #34013 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34013/consoleFull) for PR 6591 at commit [`fae9c0c`](https://gith

[GitHub] spark pull request: [Spark-7983] [MLlib] Add require for one-based...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6538#issuecomment-108065123 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 project does n

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108064953 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 h

[GitHub] spark pull request: just for testing ...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108064975 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: just for testing ...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108064944 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 h

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread MechCoder
Github user MechCoder commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108065006 lol. sorry about the silly mistake. --- 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 doe

[GitHub] spark pull request: [SPARK-7529] [mllib] Added several Java-friend...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6562#issuecomment-108065046 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 project

[GitHub] spark pull request: [SPARK-7969] [SQL] Added a DataFrame.drop func...

2015-06-02 Thread ogirardot
Github user ogirardot commented on the pull request: https://github.com/apache/spark/pull/6585#issuecomment-108064219 Can you add a test with a join 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

[GitHub] spark pull request: just for testing ...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108064317 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 project

[GitHub] spark pull request: just for testing ...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108064311 [Test build #34012 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34012/consoleFull) for PR 6591 at commit [`9f01d7e`](https://git

[GitHub] spark pull request: just for testing ...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108063632 [Test build #34012 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34012/consoleFull) for PR 6591 at commit [`9f01d7e`](https://gith

[GitHub] spark pull request: just for testing ...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108063490 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: just for testing ...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108063445 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 h

[GitHub] spark pull request: [SPARK-7969] [SQL] Added a DataFrame.drop func...

2015-06-02 Thread ogirardot
Github user ogirardot commented on the pull request: https://github.com/apache/spark/pull/6585#issuecomment-108063274 @rxin @marmbrus can you please review this PR ? --- 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-8032] [PySpark] Make version checking f...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108062736 [Test build #34011 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34011/consoleFull) for PR 6579 at commit [`893fb7e`](https://gith

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108062942 @MechCoder please test it on your local and watch the Jenkins error message. For example, In https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/

[GitHub] spark pull request: just for testing ...

2015-06-02 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108062520 lol, nice touch @JoshRosen (btw, its OK to use jenkins to run tests for me like this as long I dont' go overboard, right?) --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108061864 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 h

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108061914 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: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread MechCoder
Github user MechCoder commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108061216 I see, I misunderstood the comment '"MLlib requires NumPy 1.4+" --- 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-7387][ml][doc] CrossValidator example c...

2015-06-02 Thread harsha2010
Github user harsha2010 commented on the pull request: https://github.com/apache/spark/pull/6358#issuecomment-108060699 @jkbradley yes it does much better now Row(id=4, text=u'spark i j k', probability=DenseVector([0.248, 0.752]), prediction=1.0) Row(id=5, text=u'l m n', p

[GitHub] spark pull request: [SPARK-7785] [MLlib] [PySpark] Add __str__ and...

2015-06-02 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/6342#issuecomment-108060709 LGTM. @mengxr Could you take a look at this one? --- 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-8032] [PySpark] Make version checking f...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108060192 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 project

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108060118 [Test build #34003 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34003/consoleFull) for PR 6579 at commit [`e89376c`](https://git

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108060181 @MechCoder Why it is `<=` instead of `<`? We do support numpy 1.4.x like 1.4.1. --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/6579#discussion_r31559102 --- Diff: python/pyspark/mllib/__init__.py --- @@ -23,7 +23,14 @@ # MLlib currently needs NumPy 1.4+, so complain if lower import numpy -if

[GitHub] spark pull request: [SPARK-7547][ML] Scala Example code for Elasti...

2015-06-02 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/6576#discussion_r31558278 --- Diff: examples/src/main/scala/org/apache/spark/examples/ml/LogisticRegressionExample.scala --- @@ -0,0 +1,160 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-7529] [mllib] Added several Java-friend...

2015-06-02 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/6562#issuecomment-108058452 LGTM pending Jenkins. --- 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 f

[GitHub] spark pull request: [SPARK-7956] [SQL] Use Janino to compile SQL e...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6479#issuecomment-108056298 [Test build #34010 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34010/consoleFull) for PR 6479 at commit [`f37c8c3`](https://gith

[GitHub] spark pull request: [SPARK-7529] [mllib] Added several Java-friend...

2015-06-02 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/6562#discussion_r31558805 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/clustering/LDAModel.scala --- @@ -345,6 +346,12 @@ class DistributedLDAModel private ( }

[GitHub] spark pull request: [SPARK-7387][ml][doc] CrossValidator example c...

2015-06-02 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/6358#issuecomment-108056706 @harsha2010 Thanks! (Does it give the hoped-for predictions on the test data now when you run it?) --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-7547][ML] Scala Example code for Elasti...

2015-06-02 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/6576#issuecomment-108056478 Looks good other than those small comments. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-7547][ML] Scala Example code for Elasti...

2015-06-02 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/6576#discussion_r31558270 --- Diff: examples/src/main/scala/org/apache/spark/examples/ml/LinearRegressionExample.scala --- @@ -0,0 +1,149 @@ +/* + * Licensed to the Apache S

[GitHub] spark pull request: just for testing ...

2015-06-02 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108056329 ![hohollywoodbungalow4907imgsourcesunderconstruction](https://cloud.githubusercontent.com/assets/50748/7944291/9c0b3ad8-091e-11e5-9f3d-cc5b5c6e50f7.gif) ![ararea51

[GitHub] spark pull request: [SPARK-7547][ML] Scala Example code for Elasti...

2015-06-02 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/6576#discussion_r31558267 --- Diff: examples/src/main/scala/org/apache/spark/examples/ml/LinearRegressionExample.scala --- @@ -0,0 +1,149 @@ +/* + * Licensed to the Apache S

[GitHub] spark pull request: [SPARK-7956] [SQL] Use Janino to compile SQL e...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6479#issuecomment-108055732 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 h

[GitHub] spark pull request: [SPARK-7956] [SQL] Use Janino to compile SQL e...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6479#issuecomment-108055798 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: [Spark-7780][MLLIB] Intercept in logisticregre...

2015-06-02 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/6386#issuecomment-108054234 @dbtsai : would it be reasonable to just run the tests in databricks/spark-perf#72 instead of waiting on getting it merged? --- If your project is set up for it, you ca

[GitHub] spark pull request: [SPARK-7956] [SQL] Use Janino to compile SQL e...

2015-06-02 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/6479#discussion_r31557675 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateProjection.scala --- @@ -38,201 +42,191 @@ object GenerateProj

[GitHub] spark pull request: [SPARK-8038] [SQL] [PYSPARK] fix Column.when()...

2015-06-02 Thread ogirardot
Github user ogirardot commented on the pull request: https://github.com/apache/spark/pull/6590#issuecomment-108052390 Thanks for taking over :) --- 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-8038] [SQL] [PYSPARK] fix Column.when()...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6590#issuecomment-108050513 [Test build #34009 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34009/consoleFull) for PR 6590 at commit [`c0f2069`](https://gith

[GitHub] spark pull request: [SPARK-8038] [SQL] [PYSPARK] fix Column.when()...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6590#issuecomment-108049709 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 h

[GitHub] spark pull request: [SPARK-8038] [SQL] [PYSPARK] fix Column.when()...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6590#issuecomment-108049760 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: just for testing ...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108049445 [Test build #34007 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34007/consoleFull) for PR 6591 at commit [`b5d8ec5`](https://git

[GitHub] spark pull request: just for testing ...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108049450 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 project

[GitHub] spark pull request: just for testing ...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108048809 [Test build #34007 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34007/consoleFull) for PR 6591 at commit [`b5d8ec5`](https://gith

[GitHub] spark pull request: [SPARK-7387][ml][doc] CrossValidator example c...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6358#issuecomment-108048876 [Test build #34008 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34008/consoleFull) for PR 6358 at commit [`63efda2`](https://gith

[GitHub] spark pull request: [SPARK-7161][History Server] Provide REST api ...

2015-06-02 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5792#discussion_r31555906 --- Diff: core/src/test/scala/org/apache/spark/deploy/history/FsHistoryProviderSuite.scala --- @@ -335,6 +337,53 @@ class FsHistoryProviderSuite extends FunSu

[GitHub] spark pull request: [SPARK-7161][History Server] Provide REST api ...

2015-06-02 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/5792#issuecomment-108048755 LGTM, just a few minor things. --- 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 ha

[GitHub] spark pull request: [SPARK-8014] [SQL] Avoid premature metadata di...

2015-06-02 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/6583#issuecomment-108048336 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 and

[GitHub] spark pull request: [SPARK-8038] [SQL] [PYSPARK] fix Column.when()...

2015-06-02 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/6590#issuecomment-108048107 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 t

[GitHub] spark pull request: [SPARK-7387][ml][doc] CrossValidator example c...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6358#issuecomment-108047897 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 h

[GitHub] spark pull request: [SPARK-7387][ml][doc] CrossValidator example c...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6358#issuecomment-108047946 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: just for testing ...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108047886 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 h

[GitHub] spark pull request: just for testing ...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6591#issuecomment-108047919 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: [SPARK-7161][History Server] Provide REST api ...

2015-06-02 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5792#discussion_r31555464 --- Diff: core/src/test/scala/org/apache/spark/deploy/history/HistoryServerSuite.scala --- @@ -147,6 +148,86 @@ class HistoryServerSuite extends FunSuite with

[GitHub] spark pull request: [SPARK-8038] [SQL] [PYSPARK] fix Column.when()...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6590#issuecomment-108047381 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 project

[GitHub] spark pull request: [SPARK-8038] [SQL] [PYSPARK] fix Column.when()...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6590#issuecomment-108047348 [Test build #34002 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34002/consoleFull) for PR 6590 at commit [`c0f2069`](https://git

[GitHub] spark pull request: [SPARK-7161][History Server] Provide REST api ...

2015-06-02 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5792#discussion_r31555285 --- Diff: core/src/test/scala/org/apache/spark/deploy/history/FsHistoryProviderSuite.scala --- @@ -335,6 +337,53 @@ class FsHistoryProviderSuite extends FunSu

[GitHub] spark pull request: [SPARK-7387][ml][doc] CrossValidator example c...

2015-06-02 Thread harsha2010
Github user harsha2010 commented on the pull request: https://github.com/apache/spark/pull/6358#issuecomment-108046949 @jkbradley thanks, i have updated the cross validator with enough examples now --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-4118] [MLlib] [PySpark] Python bindings...

2015-06-02 Thread MechCoder
Github user MechCoder commented on a diff in the pull request: https://github.com/apache/spark/pull/6499#discussion_r31555361 --- Diff: python/pyspark/mllib/tests.py --- @@ -818,6 +830,78 @@ def test_model_transform(self): self.assertEqual(model.transform([1.0, 2.0, 3.0

[GitHub] spark pull request: [SPARK-8014] [SQL] Avoid premature metadata di...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6583#issuecomment-108046460 [Test build #34006 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34006/consoleFull) for PR 6583 at commit [`1aafabd`](https://gith

[GitHub] spark pull request: [SPARK-7161][History Server] Provide REST api ...

2015-06-02 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5792#discussion_r31555227 --- Diff: core/src/test/scala/org/apache/spark/deploy/history/FsHistoryProviderSuite.scala --- @@ -335,6 +337,53 @@ class FsHistoryProviderSuite extends FunSu

[GitHub] spark pull request: [SPARK-8014] [SQL] Avoid premature metadata di...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6583#issuecomment-108046188 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 h

[GitHub] spark pull request: just for testing ...

2015-06-02 Thread squito
GitHub user squito opened a pull request: https://github.com/apache/spark/pull/6591 just for testing ... don't look at this yet, just running some tests -- I'll update title / description when its ready to look at. You can merge this pull request into a Git repository by running:

[GitHub] spark pull request: [SPARK-8014] [SQL] Avoid premature metadata di...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6583#issuecomment-108046222 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: [SPARK-7161][History Server] Provide REST api ...

2015-06-02 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5792#discussion_r31554863 --- Diff: core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala --- @@ -219,6 +221,67 @@ private[history] class FsHistoryProvider(conf:

[GitHub] spark pull request: [SPARK-6324] [core] Centralize handling of scr...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5841#issuecomment-108043870 **[Test build #33996 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33996/consoleFull)** for PR 5841 at commit [`2821481`](https://gi

[GitHub] spark pull request: [SPARK-7555][docs] Add doc for elastic net in ...

2015-06-02 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/6504#issuecomment-108044100 @coderxiang One more comment: Could you please mention the phrases "Lasso" and "Ridge Regression" in the spark.ml doc? E.g., say how to set elasticNetParam to get Las

[GitHub] spark pull request: [SPARK-6324] [core] Centralize handling of scr...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5841#issuecomment-108043914 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 project

[GitHub] spark pull request: [SPARK-8015] [flume] Remove Guava dependency f...

2015-06-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/6555 --- 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-7387][ml][doc] CrossValidator example c...

2015-06-02 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/6358#issuecomment-108040143 Except for the issue with needing more training examples, this looks good to me. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-5479] [yarn] Handle --py-files correctl...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6360#issuecomment-108039612 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 project

[GitHub] spark pull request: [SPARK-5479] [yarn] Handle --py-files correctl...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6360#issuecomment-108039565 **[Test build #33995 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33995/consoleFull)** for PR 6360 at commit [`220358b`](https://gi

[GitHub] spark pull request: [SPARK-7529] [mllib] Added several Java-friend...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6562#issuecomment-108038589 [Test build #34005 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34005/consoleFull) for PR 6562 at commit [`c16821b`](https://gith

[GitHub] spark pull request: [SPARK-7387][ml][doc] CrossValidator example c...

2015-06-02 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/6358#discussion_r31552943 --- Diff: examples/src/main/python/ml/cross_validator.py --- @@ -0,0 +1,87 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or mor

[GitHub] spark pull request: [SPARK-7529] [mllib] Added several Java-friend...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6562#issuecomment-108037911 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: [SPARK-7529] [mllib] Added several Java-friend...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6562#issuecomment-108037814 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 h

[GitHub] spark pull request: [SPARK-7529] [mllib] Added several Java-friend...

2015-06-02 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/6562#discussion_r31552762 --- Diff: mllib/src/test/java/org/apache/spark/mllib/stat/JavaStatisticsSuite.java --- @@ -0,0 +1,56 @@ +/* + * Licensed to the Apache Software Foun

[GitHub] spark pull request: [SPARK-8027] [SPARKR] Add maven profile to bui...

2015-06-02 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/6567#discussion_r31552636 --- Diff: dev/create-release/create-release.sh --- @@ -228,14 +228,14 @@ if [[ ! "$@" =~ --skip-package ]]; then # We increment the Zinc port eac

[GitHub] spark pull request: [SPARK-7529] [mllib] Added several Java-friend...

2015-06-02 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/6562#discussion_r31552623 --- Diff: mllib/src/test/java/org/apache/spark/ml/param/JavaTestParams.java --- @@ -72,15 +72,31 @@ public JavaTestParams setMyStringParam(String value) {

[GitHub] spark pull request: [SPARK-7529] [mllib] Added several Java-friend...

2015-06-02 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/6562#discussion_r31552366 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/clustering/LDAModel.scala --- @@ -345,6 +346,12 @@ class DistributedLDAModel private ( }

[GitHub] spark pull request: [SPARK-7956] [SQL] Use Janino to compile SQL e...

2015-06-02 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/6479#discussion_r31552213 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala --- @@ -36,14 +38,12 @@ class LongHashSet extends

[GitHub] spark pull request: [SPARK-8015] [flume] Remove Guava dependency f...

2015-06-02 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/6555#issuecomment-108035561 LGTM. Merging this, in master and branch 1.4. --- 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 projec

[GitHub] spark pull request: [SPARK-7555][docs] Add doc for elastic net in ...

2015-06-02 Thread coderxiang
Github user coderxiang commented on the pull request: https://github.com/apache/spark/pull/6504#issuecomment-108035368 @dbtsai sounds good. Let me start the your Scala code. --- 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-8032] [PySpark] Make version checking f...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108034535 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 h

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108035114 [Test build #34004 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34004/consoleFull) for PR 6579 at commit [`e35f0d4`](https://gith

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108034579 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: [SPARK-7045] [MLlib] Avoid intermediate repres...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5748#issuecomment-108029957 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 project

[GitHub] spark pull request: [SPARK-7045] [MLlib] Avoid intermediate repres...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5748#issuecomment-108029940 [Test build #33999 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33999/consoleFull) for PR 5748 at commit [`14ee596`](https://git

[GitHub] spark pull request: [WIP][SPARK-7025] Java-friendly InputSource AP...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5603#issuecomment-108029729 [Test build #33997 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33997/consoleFull) for PR 5603 at commit [`2a2f213`](https://git

[GitHub] spark pull request: [WIP][SPARK-7025] Java-friendly InputSource AP...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5603#issuecomment-108029749 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 project

[GitHub] spark pull request: [SPARK-6164] [ML] CrossValidatorModel should k...

2015-06-02 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/5915#issuecomment-108029093 Uh oh, those tests won't work because of merge conflicts. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108028256 [Test build #34003 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34003/consoleFull) for PR 6579 at commit [`e89376c`](https://gith

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread MechCoder
Github user MechCoder commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108027679 done --- 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-6164] [ML] CrossValidatorModel should k...

2015-06-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5915#issuecomment-108027840 [Test build #871 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/871/consoleFull) for PR 5915 at commit [`15d1015`](https://githu

[GitHub] spark pull request: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108027761 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: [SPARK-8032] [PySpark] Make version checking f...

2015-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6579#issuecomment-108027731 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 h

<    1   2   3   4   5   6   7   8   9   10   >