[GitHub] spark pull request: [SPARK-14345][SQL] Decouple deserializer expre...

2016-04-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12131 --- 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: [SPARK-14345][SQL] Decouple deserializer expre...

2016-04-05 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/12131#issuecomment-205918635 LGTM, thanks for improving the comments. Its much clearer to me what is happing now!] Merging to master. --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-14397][WEBUI] and tags ar...

2016-04-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12170 --- 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: [SPARK-14370][MLLIB]removed duplicate generati...

2016-04-05 Thread pravingadakh
Github user pravingadakh commented on the pull request: https://github.com/apache/spark/pull/12176#issuecomment-205918384 I could document the returned values, but frankly I have no idea what those values are. I can see the documentation of first returned value `gammad` in the

[GitHub] spark pull request: [SPARK-14397][WEBUI] and tags ar...

2016-04-05 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/12170#issuecomment-205917914 LGTM. Merging 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

[GitHub] spark pull request: [SPARK-13538][ML] Add GaussianMixture to ML

2016-04-05 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/11419#issuecomment-205916377 Haha OK thanks. I just sent a PR to update this PR: [https://github.com/zhengruifeng/spark/pull/1] --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-14296][SQL] whole stage codegen support...

2016-04-05 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/12087#discussion_r58582999 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetBenchmark.scala --- @@ -0,0 +1,79 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-14284][ML] KMeansSummary deprecating si...

2016-04-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12084 --- 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: [SPARK-13048][ML][MLLIB] keepLastCheckpoint op...

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

[GitHub] spark pull request: [SPARK-14290][CORE][Network] avoid significant...

2016-04-05 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/12083#issuecomment-205914342 @liyezhang556520 I like the idea of eargerly copying into a direct buffer, but understand that might be a lot of code for not much gain. I still think we should reduce

[GitHub] spark pull request: [SPARK-14284][ML] KMeansSummary deprecating si...

2016-04-05 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/12084#issuecomment-205914146 LGTM Merging with master 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 your

[GitHub] spark pull request: [SPARK-13048][ML][MLLIB] keepLastCheckpoint op...

2016-04-05 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/12166#discussion_r58582117 --- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/LDA.scala --- @@ -619,6 +651,31 @@ class DistributedLDAModel private[ml] (

[GitHub] spark pull request: [SPARK-13048][ML][MLLIB] keepLastCheckpoint op...

2016-04-05 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/12166#discussion_r58582132 --- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/LDA.scala --- @@ -758,6 +816,10 @@ class LDA @Since("1.6.0") ( @Since("1.6.0")

[GitHub] spark pull request: [SPARK-13048][ML][MLLIB] keepLastCheckpoint op...

2016-04-05 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/12166#issuecomment-205913961 Updated! --- 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-13048][ML][MLLIB] keepLastCheckpoint op...

2016-04-05 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/12166#discussion_r58582125 --- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/LDA.scala --- @@ -258,7 +265,30 @@ private[clustering] trait LDAParams extends Params with

[GitHub] spark pull request: [SPARK-14402][SQL] initcap UDF doesn't match H...

2016-04-05 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/12175#issuecomment-205913234 Thank you, @srowen ! --- 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-14402][SQL] initcap UDF doesn't match H...

2016-04-05 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12175#issuecomment-205910764 I think that's pretty reasonable as a minimally invasive fix. --- 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-14257][SQL]Allow multiple continuous qu...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12049#issuecomment-205910352 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-14257][SQL]Allow multiple continuous qu...

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

[GitHub] spark pull request: [SPARK-14257][SQL]Allow multiple continuous qu...

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

[GitHub] spark pull request: [WIP][SPARK-14402][CORE] initcap UDF doesn't m...

2016-04-05 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/12175#issuecomment-205902962 Hi, @srowen . I minimized the change on master. * Undo the changes on `common` module. * Implement `initCap` by the following changes. ```

[GitHub] spark pull request: [SPARK-14296][SQL] whole stage codegen support...

2016-04-05 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/12087#discussion_r58577527 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetBenchmark.scala --- @@ -0,0 +1,79 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [WIP][SPARK-14402][CORE] initcap UDF doesn't m...

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

[GitHub] spark pull request: [SPARK-14407][SQL] Hides HadoopFsRelation rela...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12178#issuecomment-205900906 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-14129][SQL] Alter table DDL commands

2016-04-05 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12121#issuecomment-205901169 I think this PR also resolves another JIRA: https://issues.apache.org/jira/browse/SPARK-14128 Thanks! --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-14407][SQL] Hides HadoopFsRelation rela...

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

[GitHub] spark pull request: [SPARK-14407][SQL] Hides HadoopFsRelation rela...

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

[GitHub] spark pull request: [SPARK-14296][SQL] whole stage codegen support...

2016-04-05 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/12087#discussion_r58576642 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetBenchmark.scala --- @@ -0,0 +1,79 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-14354][SQL] Let Expand take name expres...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12138#issuecomment-205897396 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-14354][SQL] Let Expand take name expres...

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

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-05 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-205893057 cc @andrewor14 @yhuai --- 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-14353] Dataset Time Window `window` API...

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

[GitHub] spark pull request: [SPARK-14369][SQL][test-hadoop2.2] Locality su...

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

[GitHub] spark pull request: [SPARK-12569][PySpark][ML]:DecisionTreeRegress...

2016-04-05 Thread wangmiao1981
Github user wangmiao1981 commented on the pull request: https://github.com/apache/spark/pull/12116#issuecomment-205896732 @holdenk Thanks for your comments! I will make changes accordingly. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-14407][SQL] Hides HadoopFsRelation rela...

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

[GitHub] spark pull request: [SPARK-14354][SQL] Let Expand take name expres...

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

[GitHub] spark pull request: [SPARK-14296][SQL] whole stage codegen support...

2016-04-05 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/12087#discussion_r58575785 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetBenchmark.scala --- @@ -0,0 +1,79 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-14407][SQL] Hides HadoopFsRelation rela...

2016-04-05 Thread liancheng
GitHub user liancheng opened a pull request: https://github.com/apache/spark/pull/12178 [SPARK-14407][SQL] Hides HadoopFsRelation related data source API into execution package ## What changes were proposed in this pull request? This PR moves `HadoopFsRelation` related

[GitHub] spark pull request: [SPARK-14296][SQL] whole stage codegen support...

2016-04-05 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/12087#discussion_r58575623 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetBenchmark.scala --- @@ -0,0 +1,79 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11989#issuecomment-205893715 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-14124] [SQL] [FOLLOWUP] Implement Datab...

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

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

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

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-05 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-205893108 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

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

[GitHub] spark pull request: [SPARK-14396] [SQL] Throw Exceptions for DDLs ...

2016-04-05 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12169#issuecomment-205892839 Thanks for the review! @hvanhovell Also cc @yhuai @andrewor14 --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-6429] Implement hashCode and equals tog...

2016-04-05 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/12157#discussion_r58572140 --- Diff: core/src/main/scala/org/apache/spark/rdd/HadoopRDD.scala --- @@ -53,14 +53,22 @@ import org.apache.spark.util.{NextIterator,

[GitHub] spark pull request: [SPARK-14370][MLLIB]removed duplicate generati...

2016-04-05 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12176#issuecomment-205882669 A-ha, understood about `.values`. This looks pretty reasonable. My only question is, does it make sense conceptually that this method also returns a list of IDs? it

[GitHub] spark pull request: [SPARK-14245] [Web UI] Display the user in the...

2016-04-05 Thread ajbozarth
Github user ajbozarth commented on the pull request: https://github.com/apache/spark/pull/12123#issuecomment-205882481 I'll add it the history server later today then, I didn't re-test on the history server after the change so I didn't notice. --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-10530] [CORE] Kill other task attempts ...

2016-04-05 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/11996#discussion_r58569079 --- Diff: core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala --- @@ -789,6 +791,51 @@ class TaskSetManagerSuite extends

[GitHub] spark pull request: [SPARK-14296][SQL] whole stage codegen support...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12087#issuecomment-205881415 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-14296][SQL] whole stage codegen support...

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

[GitHub] spark pull request: [SPARK-14296][SQL] whole stage codegen support...

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

[GitHub] spark pull request: [SPARK-14369][SQL][test-hadoop2.2] Locality su...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12153#issuecomment-205878760 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-3724][ML] RandomForest: More options fo...

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

[GitHub] spark pull request: [SPARK-14123] [SPARK-14384] [SQL] Handle Creat...

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

[GitHub] spark pull request: [SPARK-14369][SQL][test-hadoop2.2] Locality su...

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

[GitHub] spark pull request: [SPARK-14123] [SPARK-14384] [SQL] Handle Creat...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12117#issuecomment-205878599 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-14369][SQL][test-hadoop2.2] Locality su...

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

[GitHub] spark pull request: [SPARK-14123] [SPARK-14384] [SQL] Handle Creat...

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

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

2016-04-05 Thread yongtang
Github user yongtang commented on the pull request: https://github.com/apache/spark/pull/11989#issuecomment-205877757 @sethah Thanks. The import has been removed. --- 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-14257][SQL]Allow multiple continuous qu...

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

[GitHub] spark pull request: [SPARK-14257][SQL]Allow multiple continuous qu...

2016-04-05 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/12049#issuecomment-205869083 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-14335][SQL] Describe function command r...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12128#issuecomment-205868721 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-14335][SQL] Describe function command r...

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

[GitHub] spark pull request: [SPARK-14335][SQL] Describe function command r...

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

[GitHub] spark pull request: [SPARK-14362] [SPARK-14406] [SQL] [WIP] DDL Na...

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

[GitHub] spark pull request: [SPARK-14362] [SPARK-14406] [SQL] [WIP] DDL Na...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12146#issuecomment-205866050 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-10530] [CORE] Kill other task attempts ...

2016-04-05 Thread devaraj-kavali
Github user devaraj-kavali commented on a diff in the pull request: https://github.com/apache/spark/pull/11996#discussion_r58563479 --- Diff: core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala --- @@ -789,6 +791,51 @@ class TaskSetManagerSuite extends

[GitHub] spark pull request: [SPARK-14362] [SPARK-14406] [SQL] [WIP] DDL Na...

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

[GitHub] spark pull request: [SPARK-14362] [SPARK-14406] [SQL] [WIP] DDL Na...

2016-04-05 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12146#issuecomment-205861283 @yhuai Sure, will do it in this PR. 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 your

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

2016-04-05 Thread sethah
Github user sethah commented on the pull request: https://github.com/apache/spark/pull/11989#issuecomment-205857272 This LGTM other than one small comment about imports. @MLnick could you make a final pass? --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-14399] Remove unnecessary excludes from...

2016-04-05 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/12171#issuecomment-205858120 It looks like Hive might actually need Joda time: ``` - analyze MetastoreRelations *** FAILED ***

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

2016-04-05 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/11989#discussion_r58558671 --- Diff: mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala --- @@ -27,6 +27,7 @@ import

[GitHub] spark pull request: [SPARK-14396] [SQL] Throw Exceptions for DDLs ...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12169#issuecomment-205855786 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-14396] [SQL] Throw Exceptions for DDLs ...

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

[GitHub] spark pull request: [SPARK-14396] [SQL] Throw Exceptions for DDLs ...

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

[GitHub] spark pull request: [SPARK-14354][SQL] Let Expand take name expres...

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

[GitHub] spark pull request: [SPARK-14354][SQL] Let Expand take name expres...

2016-04-05 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/12138#issuecomment-205851532 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-14296][SQL] whole stage codegen support...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12087#issuecomment-205850589 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-14296][SQL] whole stage codegen support...

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

[GitHub] spark pull request: [SPARK-14296][SQL] whole stage codegen support...

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

[GitHub] spark pull request: [SPARK-14296][SQL] whole stage codegen support...

2016-04-05 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/12087#issuecomment-205848733 benchmark added, the result is included in the benchmark code. I also ran this benchmark against master branch, the result is: ``` Java HotSpot(TM) 64-Bit

[GitHub] spark pull request: [SPARK-14354][SQL] Let Expand take name expres...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12138#issuecomment-205848612 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-14354][SQL] Let Expand take name expres...

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

[GitHub] spark pull request: [SPARK-14354][SQL] Let Expand take name expres...

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

[GitHub] spark pull request: [SPARK-14296][SQL] whole stage codegen support...

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

[GitHub] spark pull request: [SPARK-14370][MLLIB]removed duplicate generati...

2016-04-05 Thread pravingadakh
Github user pravingadakh commented on a diff in the pull request: https://github.com/apache/spark/pull/12176#discussion_r58553918 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/clustering/LDAOptimizer.scala --- @@ -542,10 +539,11 @@ private[clustering] object

[GitHub] spark pull request: [SPARK-14354][SQL] Let Expand take name expres...

2016-04-05 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/12138#discussion_r58553204 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -1659,11 +1665,12 @@ object TimeWindowing extends

[GitHub] spark pull request: [SPARK-14370][MLLIB]removed duplicate generati...

2016-04-05 Thread pravingadakh
Github user pravingadakh commented on a diff in the pull request: https://github.com/apache/spark/pull/12176#discussion_r58553056 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/clustering/LDAOptimizer.scala --- @@ -440,12 +440,9 @@ final class OnlineLDAOptimizer extends

[GitHub] spark pull request: [SPARK-14354][SQL] Let Expand take name expres...

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

[GitHub] spark pull request: [SPARK-12566] [ML] [WIP] GLM model family, lin...

2016-04-05 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/11549#issuecomment-205844978 @jkbradley @hhbyyh I can work on 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-14245] [Web UI] Display the user in the...

2016-04-05 Thread sarutak
Github user sarutak commented on the pull request: https://github.com/apache/spark/pull/12123#issuecomment-205844392 Yeah, we can see the user name in the history page and it's gotten using the REST API. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-14298] [ML] [MLlib] LDA should support ...

2016-04-05 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/12089#issuecomment-205843565 @jkbradley Agree, I will update the PR soon. --- 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-14369][SQL][WIP][test-hadoop2.2] Locali...

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

[GitHub] spark pull request: [SPARK-14404][SQL][TESTS] HDFSMetadataLogSuite...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12177#issuecomment-205837805 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-14204] [SQL] register driverClass rathe...

2016-04-05 Thread mchalek
Github user mchalek commented on the pull request: https://github.com/apache/spark/pull/12000#issuecomment-205840895 Bump. Would be nice to get closure on this. I doubt that we were in the minority in being affected by this (although admittedly, complaints of other failures seem

[GitHub] spark pull request: [SPARK-14296][SQL] whole stage codegen support...

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

[GitHub] spark pull request: [SPARK-14362] [SQL] DDL Native Support: Drop V...

2016-04-05 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/12146#issuecomment-205839654 @gatorsmile Thank you for working on it. I feel changes of DropTable command should also make this command natively supported by Spark (no runSqlHive)? If so, we can

[GitHub] spark pull request: [SPARK-14296][SQL] whole stage codegen support...

2016-04-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12087#issuecomment-205839287 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-14296][SQL] whole stage codegen support...

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

<    2   3   4   5   6   7   8   9   10   >