[GitHub] spark pull request: [SPARK-12469][CORE][RFC/WIP] Add Consistent Ac...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10841#issuecomment-173537102 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-12469][CORE][RFC/WIP] Add Consistent Ac...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10841#issuecomment-173537104 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173538635 **[Test build #49872 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49872/consoleFull)** for PR 10723 at commit

[GitHub] spark pull request: [SPARK-9835] [ML] IterativelyReweightedLeastSq...

2016-01-21 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/10639#discussion_r50390322 --- Diff: mllib/src/main/scala/org/apache/spark/ml/glm/Families.scala --- @@ -0,0 +1,138 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-9835] [ML] IterativelyReweightedLeastSq...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10639#issuecomment-173545758 **[Test build #49875 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49875/consoleFull)** for PR 10639 at commit

[GitHub] spark pull request: [SPARK-12401][SQL] Add integration tests for p...

2016-01-21 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/10596#issuecomment-173553803 @liancheng @yhuai ping --- 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-6106:Support user group mapping and grou...

2016-01-21 Thread alpivonka
Github user alpivonka commented on the pull request: https://github.com/apache/spark/pull/5325#issuecomment-173561257 I would like to bring up an opportunity for re-use. with in the mapred-site.xml (for our implementation) we are using the following to allow access to mapreduce

[GitHub] spark pull request: [SPARK-11327] [MESOS] Dispatcher does not resp...

2016-01-21 Thread dragos
Github user dragos commented on a diff in the pull request: https://github.com/apache/spark/pull/10370#discussion_r50402728 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosClusterScheduler.scala --- @@ -440,6 +446,9 @@ private[spark] class

[GitHub] spark pull request: [SPARK-11780][SQL] Add type aliases backwards ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10635#issuecomment-173579793 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-11780][SQL] Add type aliases backwards ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10635#issuecomment-173579799 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-9835] [ML] IterativelyReweightedLeastSq...

2016-01-21 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/10639#discussion_r50390974 --- Diff: mllib/src/main/scala/org/apache/spark/ml/optim/IterativelyReweightedLeastSquares.scala --- @@ -0,0 +1,101 @@ +/* + * Licensed to the

[GitHub] spark pull request: [SPARK-9835] [ML] IterativelyReweightedLeastSq...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10639#issuecomment-173554879 **[Test build #49875 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49875/consoleFull)** for PR 10639 at commit

[GitHub] spark pull request: [SPARK-9835] [ML] IterativelyReweightedLeastSq...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10639#issuecomment-173554998 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-9835] [ML] IterativelyReweightedLeastSq...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10639#issuecomment-173554996 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-11780][SQL] Add type aliases backwards ...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10635#issuecomment-173556798 **[Test build #49876 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49876/consoleFull)** for PR 10635 at commit

[GitHub] spark pull request: [SPARK-9835] [ML] IterativelyReweightedLeastSq...

2016-01-21 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/10639#issuecomment-173548548 @mengxr Thanks for your comments. For the issue that ```WeightedLeastSquares``` contains extra content such as ```diagInvAWA```, it will be used to generate

[GitHub] spark pull request: [SPARK-12686][SQL] Support group-by push down ...

2016-01-21 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/10631#issuecomment-173552264 @rxin ping --- 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-12476][SQL] Implement JdbcRelation#unha...

2016-01-21 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/10427#issuecomment-173552368 @yhuai ping --- 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-2827][GraphX] Add collectDegreeDist to ...

2016-01-21 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/10521#issuecomment-173553650 @andrewor14 ping --- 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-11780][SQL] Add type aliases backwards ...

2016-01-21 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/10635#issuecomment-173553404 @marmbrus ping --- 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-10648] Oracle dialect to handle nonspec...

2016-01-21 Thread dsdinter
Github user dsdinter commented on the pull request: https://github.com/apache/spark/pull/9495#issuecomment-173553569 It seems this issue in OJDBC and started to happen after Oracle 11g:

[GitHub] spark pull request: [SPARK-12469][CORE][RFC/WIP] Add Consistent Ac...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10841#issuecomment-173536939 **[Test build #49873 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49873/consoleFull)** for PR 10841 at commit

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173538837 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173538836 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-11780][SQL] Add type aliases backwards ...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10635#issuecomment-173579529 **[Test build #49876 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49876/consoleFull)** for PR 10635 at commit

[GitHub] spark pull request: [SPARK-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10788#discussion_r50369722 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala --- @@ -335,31 +342,45 @@ class LogisticRegression

[GitHub] spark pull request: [SPARK-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10788#discussion_r50369730 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala --- @@ -374,11 +395,11 @@ class LogisticRegression

[GitHub] spark pull request: [SPARK-12952] EMLDAOptimizer initialize() shou...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10863#issuecomment-173492042 **[Test build #49867 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49867/consoleFull)** for PR 10863 at commit

[GitHub] spark pull request: [SPARK-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/10788#discussion_r50372397 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala --- @@ -335,31 +342,45 @@ class LogisticRegression

[GitHub] spark pull request: [SPARK-12908][ML] Add warning message for Logi...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10862#issuecomment-173498061 **[Test build #49869 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49869/consoleFull)** for PR 10862 at commit

[GitHub] spark pull request: [SPARK-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10788#issuecomment-173506673 **[Test build #49871 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49871/consoleFull)** for PR 10788 at commit

[GitHub] spark pull request: [SPARK-12908][ML] Add warning message for Logi...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10862#issuecomment-173511498 **[Test build #49869 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49869/consoleFull)** for PR 10862 at commit

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173525273 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173525271 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-12789]Support order by index

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10731#issuecomment-173493053 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-12789]Support order by index

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10731#issuecomment-173493055 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12789]Support order by index

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10731#issuecomment-173492642 **[Test build #49864 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49864/consoleFull)** for PR 10731 at commit

[GitHub] spark pull request: [SPARK-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/10788#discussion_r50372852 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/classification/LogisticRegression.scala --- @@ -374,4 +384,82 @@ class LogisticRegressionWithLBFGS

[GitHub] spark pull request: [SPARK-12469][CORE][RFC/WIP] Add Consistent Ac...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10841#issuecomment-173506101 **[Test build #49873 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49873/consoleFull)** for PR 10841 at commit

[GitHub] spark pull request: [SPARK-10524][ML] Use the soft prediction to o...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8734#issuecomment-173506110 **[Test build #49874 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49874/consoleFull)** for PR 8734 at commit

[GitHub] spark pull request: [SPARK-12953][Examples]RDDRelation writer set ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10864#issuecomment-173506209 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-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10788#discussion_r50370169 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/classification/LogisticRegression.scala --- @@ -374,4 +384,82 @@ class LogisticRegressionWithLBFGS

[GitHub] spark pull request: [SPARK-12908][ML] Add warning message for Logi...

2016-01-21 Thread dbtsai
Github user dbtsai commented on the pull request: https://github.com/apache/spark/pull/10862#issuecomment-173494166 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

[GitHub] spark pull request: [SPARK-10524][ML] Use the soft prediction to o...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8734#issuecomment-173518885 **[Test build #49874 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49874/consoleFull)** for PR 8734 at commit

[GitHub] spark pull request: [SPARK-10524][ML] Use the soft prediction to o...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8734#issuecomment-173519027 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-10524][ML] Use the soft prediction to o...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8734#issuecomment-173519029 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12755][CORE] Stop the event logger befo...

2016-01-21 Thread mallman
Github user mallman commented on the pull request: https://github.com/apache/spark/pull/10700#issuecomment-173490509 Here are my current thoughts. Josh says this functionality is going to be removed in Spark 2.0. The bug this PR is designed to address manifests itself in Spark 1.5 in

[GitHub] spark pull request: [SPARK-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10788#discussion_r50370273 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/classification/LogisticRegression.scala --- @@ -374,4 +384,82 @@ class LogisticRegressionWithLBFGS

[GitHub] spark pull request: [SPARK-12757][WIP] Use reference counting to p...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10705#issuecomment-173490560 **[Test build #49861 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49861/consoleFull)** for PR 10705 at commit

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173494001 **[Test build #49868 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49868/consoleFull)** for PR 10723 at commit

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173507314 **[Test build #49872 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49872/consoleFull)** for PR 10723 at commit

[GitHub] spark pull request: [SPARK-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10788#issuecomment-173519318 **[Test build #49871 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49871/consoleFull)** for PR 10788 at commit

[GitHub] spark pull request: [SPARK-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10788#issuecomment-173519472 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12469][CORE][RFC/WIP] Add Consistent Ac...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10841#issuecomment-173525219 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173525124 **[Test build #49868 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49868/consoleFull)** for PR 10723 at commit

[GitHub] spark pull request: [SPARK-12469][CORE][RFC/WIP] Add Consistent Ac...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10841#issuecomment-173525217 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-12469][CORE][RFC/WIP] Add Consistent Ac...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10841#issuecomment-173525109 **[Test build #49870 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49870/consoleFull)** for PR 10841 at commit

[GitHub] spark pull request: [SPARK-7997][Core]Remove Akka from Spark Core ...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10854#issuecomment-173527896 **[Test build #49860 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49860/consoleFull)** for PR 10854 at commit

[GitHub] spark pull request: [SPARK-7997][Core]Remove Akka from Spark Core ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10854#issuecomment-173527952 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-7997][Core]Remove Akka from Spark Core ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10854#issuecomment-173527954 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12904][SQL] Strength reduction for inte...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10845#issuecomment-173491009 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12904][SQL] Strength reduction for inte...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10845#issuecomment-173491008 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-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10788#discussion_r50370414 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/classification/LogisticRegression.scala --- @@ -374,4 +384,82 @@ class LogisticRegressionWithLBFGS

[GitHub] spark pull request: [SPARK-12904][SQL] Strength reduction for inte...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10845#issuecomment-173490658 **[Test build #49863 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49863/consoleFull)** for PR 10845 at commit

[GitHub] spark pull request: [SPARK-12757][WIP] Use reference counting to p...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10705#issuecomment-173490745 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread dbtsai
Github user dbtsai commented on the pull request: https://github.com/apache/spark/pull/10788#issuecomment-173493541 LGTM except some styling issues, and concern about caching twice. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/10788#discussion_r50371017 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/classification/LogisticRegression.scala --- @@ -374,4 +384,82 @@ class LogisticRegressionWithLBFGS

[GitHub] spark pull request: [SPARK-12469][CORE][RFC/WIP] Add Consistent Ac...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10841#issuecomment-173500763 **[Test build #49870 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49870/consoleFull)** for PR 10841 at commit

[GitHub] spark pull request: [SPARK-12755][CORE] Stop the event logger befo...

2016-01-21 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10700#issuecomment-173505157 Are there downsides to merging this to master, even if the related functionality is about to be removed? it passes tests, and seems to improve an ordering of shutdown,

[GitHub] spark pull request: [SPARK-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/10788#discussion_r50372566 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala --- @@ -335,31 +342,45 @@ class LogisticRegression

[GitHub] spark pull request: fix error when run RDDRelation.main():"path fi...

2016-01-21 Thread shijinkui
GitHub user shijinkui opened a pull request: https://github.com/apache/spark/pull/10864 fix error when run RDDRelation.main():"path file:/Users/sjk/pair.parq… https://issues.apache.org/jira/browse/SPARK-12953 fix error when run RDDRelation.main(): "path

[GitHub] spark pull request: [SPARK-12908][ML] Add warning message for Logi...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10862#issuecomment-173511839 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12908][ML] Add warning message for Logi...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10862#issuecomment-173511835 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-7780][MLLIB] intercept in logisticregre...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10788#issuecomment-173519471 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-12952] EMLDAOptimizer initialize() shou...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10863#issuecomment-173492160 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-12952] EMLDAOptimizer initialize() shou...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10863#issuecomment-173492161 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12153][SPARK-7617][MLlib]add support of...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10152#issuecomment-173607998 **[Test build #2431 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2431/consoleFull)** for PR 10152 at commit

[GitHub] spark pull request: improved error message for java type inference...

2016-01-21 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10865#issuecomment-173608093 (Link this to your JIRA -- see guidance here first for how to open a PR: https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark ) --- If your project

[GitHub] spark pull request: [SPARK-12534][DOC] update documentation to lis...

2016-01-21 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10491#issuecomment-173606818 Merged to master --- 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: improved error message for java type inference...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10865#issuecomment-173608775 **[Test build #2432 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2432/consoleFull)** for PR 10865 at commit

[GitHub] spark pull request: improved error message for java type inference...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10865#issuecomment-173608886 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-12760] [DOCS] inaccurate description fo...

2016-01-21 Thread srowen
GitHub user srowen opened a pull request: https://github.com/apache/spark/pull/10866 [SPARK-12760] [DOCS] inaccurate description for difference between local vs cluster mode in closure handling Clarify that modifying a driver local variable won't have the desired effect in cluster

[GitHub] spark pull request: [SPARK-11137][Streaming] Make StreamingContext...

2016-01-21 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10807#issuecomment-173609582 @felixcheung WDYT? --- 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: improved error message for java type inference...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10865#issuecomment-173609383 **[Test build #2432 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2432/consoleFull)** for PR 10865 at commit

[GitHub] spark pull request: SPARK-11565 Replace deprecated DigestUtils.sha...

2016-01-21 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/9532#issuecomment-173609858 @gliptak are you able to follow up on this? --- 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-12534][DOC] update documentation to lis...

2016-01-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10491 --- 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: improved error message for java type inference...

2016-01-21 Thread andygrove
GitHub user andygrove opened a pull request: https://github.com/apache/spark/pull/10865 improved error message for java type inference failure You can merge this pull request into a Git repository by running: $ git pull https://github.com/codefutures/spark SPARK-12932

[GitHub] spark pull request: [SPARK-12153][SPARK-7617][MLlib]add support of...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10152#issuecomment-173621659 **[Test build #2431 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2431/consoleFull)** for PR 10152 at commit

[GitHub] spark pull request: [SPARK-12760] [DOCS] inaccurate description fo...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10866#issuecomment-173623032 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12760] [DOCS] invalid lambda expression...

2016-01-21 Thread mortada
GitHub user mortada opened a pull request: https://github.com/apache/spark/pull/10867 [SPARK-12760] [DOCS] invalid lambda expression in python example for … …local vs cluster @srowen thanks for the PR at https://github.com/apache/spark/pull/10866! sorry it took me a

[GitHub] spark pull request: [SPARK-12760] [DOCS] invalid lambda expression...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10867#issuecomment-173643528 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12760] [DOCS] invalid lambda expression...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10867#issuecomment-173643525 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-10873] Support column sort and search f...

2016-01-21 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/10648#issuecomment-173630521 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

[GitHub] spark pull request: [SPARK-12760] [DOCS] invalid lambda expression...

2016-01-21 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10867#issuecomment-173635004 Does it still execute without error on a cluster? (even if it doesn't actually increment the counter in the way someone might expect.) Certainly if it doesn't compile

[GitHub] spark pull request: [SPARK-10873] Support column sort and search f...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10648#issuecomment-173632677 **[Test build #49878 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49878/consoleFull)** for PR 10648 at commit

[GitHub] spark pull request: [HOTFIX][BUILD][TEST-MAVEN]Remove duplicate de...

2016-01-21 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/10868#issuecomment-173638249 CC @JoshRosen --- 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-12760] [DOCS] invalid lambda expression...

2016-01-21 Thread mortada
Github user mortada commented on the pull request: https://github.com/apache/spark/pull/10867#issuecomment-173648674 @srowen I tested the python code in cluster mode (5 ec2 workers) and this works fine ``` 16/01/21 17:33:29 INFO BlockManagerMasterEndpoint: Registering

[GitHub] spark pull request: [SPARK-12760] [DOCS] inaccurate description fo...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10866#issuecomment-173619459 **[Test build #49877 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49877/consoleFull)** for PR 10866 at commit

[GitHub] spark pull request: [SPARK-12760] [DOCS] inaccurate description fo...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10866#issuecomment-173623028 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-12760] [DOCS] inaccurate description fo...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10866#issuecomment-173622876 **[Test build #49877 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49877/consoleFull)** for PR 10866 at commit

  1   2   3   4   >