[GitHub] spark pull request: [SPARK-7232] [SQL] Add a Substitution batch fo...

2015-04-29 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/5776#issuecomment-97682532 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 enabled a

[GitHub] spark pull request: [SPARK-7269] [SQL] Incorrect analysis for aggr...

2015-04-29 Thread scwf
Github user scwf commented on a diff in the pull request: https://github.com/apache/spark/pull/5798#discussion_r29407019 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/package.scala --- @@ -29,12 +29,23 @@ package object analysis { /**

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29407007 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/stat/FrequentItems.scala --- @@ -0,0 +1,127 @@ +/* +* Licensed to the Apache Software F

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5799#issuecomment-97681270 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/313

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29406990 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/stat/FrequentItems.scala --- @@ -0,0 +1,127 @@ +/* +* Licensed to the Apache Software F

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5799#issuecomment-97681263 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-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29406968 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/stat/FrequentItems.scala --- @@ -0,0 +1,127 @@ +/* +* Licensed to the Apache Software F

[GitHub] spark pull request: [SPARK-7269] [SQL] Incorrect analysis for aggr...

2015-04-29 Thread scwf
Github user scwf commented on a diff in the pull request: https://github.com/apache/spark/pull/5798#discussion_r29406960 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveResolutionSuite.scala --- @@ -81,9 +81,13 @@ class HiveResolutionSuite extends HiveCom

[GitHub] spark pull request: [SPARK-7269] [SQL] Incorrect analysis for aggr...

2015-04-29 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/5798#discussion_r29406952 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/package.scala --- @@ -29,12 +29,23 @@ package object analysis {

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29406904 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameStatFunctions.scala --- @@ -0,0 +1,55 @@ +/* +* Licensed to the Apache Software Foundatio

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29406864 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameStatSuite.scala --- @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Software Foundation (

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29406886 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameStatSuite.scala --- @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Software Foundation (

[GitHub] spark pull request: [SPARK-7269] [SQL] Incorrect analysis for aggr...

2015-04-29 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/5798#discussion_r29406841 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveResolutionSuite.scala --- @@ -81,9 +81,13 @@ class HiveResolutionSuite exten

[GitHub] spark pull request: [SPARK-7232] [SQL] Add a Substitution batch fo...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5776#issuecomment-97680177 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/313

[GitHub] spark pull request: [SPARK-7232] [SQL] Add a Substitution batch fo...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5776#issuecomment-97680167 [Test build #31385 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31385/consoleFull) for PR 5776 at commit [`553005a`](https://gith

[GitHub] spark pull request: [SPARK-7232] [SQL] Add a Substitution batch fo...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5776#issuecomment-97680176 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-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29406784 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/stat/FrequentItems.scala --- @@ -0,0 +1,127 @@ +/* +* Licensed to the Apache Software F

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29406755 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameStatFunctions.scala --- @@ -0,0 +1,55 @@ +/* +* Licensed to the Apache Software Foundatio

[GitHub] spark pull request: SPARK-5112. Expose SizeEstimator as a develope...

2015-04-29 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/3913#issuecomment-97679846 Pending an update 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

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5799#issuecomment-97679720 [Test build #31392 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31392/consoleFull) for PR 5799 at commit [`482e741`](https://githu

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29406677 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameStatFunctions.scala --- @@ -0,0 +1,55 @@ +/* +* Licensed to the Apache Software Foundatio

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5799#issuecomment-97679663 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-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5799#issuecomment-97679655 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 ha

[GitHub] spark pull request: [SPARK-7269] [SQL] Incorrect analysis for aggr...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5798#issuecomment-97679301 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/313

[GitHub] spark pull request: [SPARK-7269] [SQL] Incorrect analysis for aggr...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5798#issuecomment-97679293 [Test build #31383 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31383/consoleFull) for PR 5798 at commit [`c00f1ad`](https://gith

[GitHub] spark pull request: [SPARK-7269] [SQL] Incorrect analysis for aggr...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5798#issuecomment-97679300 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-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5799#issuecomment-97679097 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 ha

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5799#issuecomment-97679106 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-7269] [SQL] Incorrect analysis for aggr...

2015-04-29 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/5798#discussion_r29406469 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveResolutionSuite.scala --- @@ -81,9 +81,13 @@ class HiveResolutionSuite extends HiveC

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29406389 --- Diff: python/pyspark/sql/dataframe.py --- @@ -1166,7 +1166,7 @@ def __init__(self, jc): # container operators __contains__ = _bin_op("

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97678166 [Test build #31390 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31390/consoleFull) for PR 5744 at commit [`51719b7`](https://githu

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97677743 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-7133][SQL] Implement struct, array, and...

2015-04-29 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97677728 @rxin , already 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 th

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97677730 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 ha

[GitHub] spark pull request: Merge pull request #1 from apache/master

2015-04-29 Thread sven0726
GitHub user sven0726 opened a pull request: https://github.com/apache/spark/pull/5800 Merge pull request #1 from apache/master You can merge this pull request into a Git repository by running: $ git pull https://github.com/sven0726/spark-1 master Alternatively you can review

[GitHub] spark pull request: Merge pull request #1 from apache/master

2015-04-29 Thread sven0726
Github user sven0726 closed the pull request at: https://github.com/apache/spark/pull/5800 --- 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 en

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97677458 I will let @marmbrus take a look at this tomorrow. Meantime, can you add the apply method and Python getitem method? Thanks. --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-1406] Mllib pmml model export

2015-04-29 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/3062#issuecomment-97677239 LGTM. Merged into master. Thanks! @selvinsource I created SPARK-7272 for the user guide and assigned it to you. Let me know if you don't have time to do it. Btw, please su

[GitHub] spark pull request: SPARK-5112. Expose SizeEstimator as a develope...

2015-04-29 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/3913#issuecomment-97676897 Looking good though this needs a rebase now. --- 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-7133][SQL] Implement struct, array, and...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97676693 [Test build #31380 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31380/consoleFull) for PR 5744 at commit [`727fbc1`](https://gith

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97676722 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-7133][SQL] Implement struct, array, and...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97676725 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/313

[GitHub] spark pull request: SPARK-6846 [WEBUI] Stage kill URL easy to acci...

2015-04-29 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/5528#issuecomment-97676514 That's fine, in the sense that the endpoint returns no data. OK, so it works except for this proxying. Hm, surely the YARN proxy can pass on a POST. We'll have to look int

[GitHub] spark pull request: [SPARK-1406] Mllib pmml model export

2015-04-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/3062 --- 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-6612] [MLLib] [PySpark] Python KMeans p...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5647#issuecomment-97675656 [Test build #31382 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31382/consoleFull) for PR 5647 at commit [`0319821`](https://gith

[GitHub] spark pull request: [SPARK-6612] [MLLib] [PySpark] Python KMeans p...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5647#issuecomment-97675701 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/313

[GitHub] spark pull request: [SPARK-6612] [MLLib] [PySpark] Python KMeans p...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5647#issuecomment-97675700 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-7120][SPARK-7121] Closure cleaner nesti...

2015-04-29 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/5685#issuecomment-97672308 Hey Andrew, This is looking good. The code is quite dense but as far as I can tell, this is correct. I left some more surface level comments, if you can get to

[GitHub] spark pull request: [SPARK-1442][SQL][WIP] Window Function Support...

2015-04-29 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/5604#issuecomment-97671446 I have been working on it a few days. I believe that my update will cover most of your comments. Please hold your comments until my update. Thanks :) --- If your project i

[GitHub] spark pull request: [SPARK-7031][ThriftServer]let thrift server ta...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5609#issuecomment-97671182 [Test build #31384 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31384/consoleFull) for PR 5609 at commit [`8d3fc16`](https://gith

[GitHub] spark pull request: [SPARK-7031][ThriftServer]let thrift server ta...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5609#issuecomment-97671186 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-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29405622 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/ml/FrequentItems.scala --- @@ -0,0 +1,124 @@ +/* +* Licensed to the Apache Software Foundation (

[GitHub] spark pull request: [SPARK-7031][ThriftServer]let thrift server ta...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5609#issuecomment-97671187 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/313

[GitHub] spark pull request: [SPARK-1442][SQL][WIP] Window Function Support...

2015-04-29 Thread guowei2
Github user guowei2 commented on the pull request: https://github.com/apache/spark/pull/5604#issuecomment-97671084 @scwf I think it is a good choice, 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 projec

[GitHub] spark pull request: [SPARK-7031][ThriftServer]let thrift server ta...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5609#issuecomment-97670982 [Test build #31389 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31389/consoleFull) for PR 5609 at commit [`8d3fc16`](https://githu

[GitHub] spark pull request: [SPARK-7031][ThriftServer]let thrift server ta...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5609#issuecomment-97670934 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-7031][ThriftServer]let thrift server ta...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5609#issuecomment-97670920 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 ha

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread brkyvz
Github user brkyvz commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29405486 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala --- @@ -1414,4 +1415,25 @@ class DataFrame private[sql]( val jrdd = rdd.map(Eval

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29405470 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/ml/FrequentItems.scala --- @@ -0,0 +1,124 @@ +/* +* Licensed to the Apache Software Foundation (

[GitHub] spark pull request: [SPARK-7031][ThriftServer]let thrift server ta...

2015-04-29 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on the pull request: https://github.com/apache/spark/pull/5609#issuecomment-97670648 Looks like the failed test case is flacky. Jenkins, retest this please. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] spark pull request: [SPARK-6602][Core] Update Master, Worker, Clie...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5392#issuecomment-97670499 [Test build #31388 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31388/consoleFull) for PR 5392 at commit [`72304f0`](https://githu

[GitHub] spark pull request: [SPARK-6602][Core] Update Master, Worker, Clie...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5392#issuecomment-97670459 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-6602][Core] Update Master, Worker, Clie...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5392#issuecomment-97670448 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 ha

[GitHub] spark pull request: [SPARK-7120][SPARK-7121] Closure cleaner nesti...

2015-04-29 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/5685#discussion_r29405393 --- Diff: core/src/test/scala/org/apache/spark/util/ClosureCleanerSuite2.scala --- @@ -0,0 +1,562 @@ +/* + * Licensed to the Apache Software Foundati

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29405344 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/ml/FrequentItems.scala --- @@ -0,0 +1,124 @@ +/* +* Licensed to the Apache Software Foundation (

[GitHub] spark pull request: [SPARK-6939][Streaming][WebUI] Add timeline an...

2015-04-29 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/5533#issuecomment-97670189 new screenshot ![g8](https://cloud.githubusercontent.com/assets/1000778/7407142/57adb2a0-eec3-11e4-8492-7304da7baaa3.png) --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-6939][Streaming][WebUI] Add timeline an...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5533#issuecomment-97670140 [Test build #31387 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31387/consoleFull) for PR 5533 at commit [`a2972e9`](https://githu

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29405307 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/ml/FrequentItems.scala --- @@ -0,0 +1,124 @@ +/* +* Licensed to the Apache Software Foundation (

[GitHub] spark pull request: [SPARK-6939][Streaming][WebUI] Add timeline an...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5533#issuecomment-97670038 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-6939][Streaming][WebUI] Add timeline an...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5533#issuecomment-97670024 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 ha

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29405254 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/ml/FrequentItems.scala --- @@ -0,0 +1,124 @@ +/* +* Licensed to the Apache Software Foundation (

[GitHub] spark pull request: [SPARK-7267][SQL]Push down Project when it's c...

2015-04-29 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/5797#issuecomment-97669998 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 feature

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29405251 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/ml/FrequentItems.scala --- @@ -0,0 +1,124 @@ +/* +* Licensed to the Apache Software Foundation (

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29405257 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/ml/FrequentItems.scala --- @@ -0,0 +1,124 @@ +/* +* Licensed to the Apache Software Foundation (

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29405043 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala --- @@ -1414,4 +1415,37 @@ class DataFrame private[sql]( val jrdd = rdd.map(Eval

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29405009 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala --- @@ -1414,4 +1415,25 @@ class DataFrame private[sql]( val jrdd = rdd.map(Evalua

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread brkyvz
Github user brkyvz commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29404981 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala --- @@ -1414,4 +1415,25 @@ class DataFrame private[sql]( val jrdd = rdd.map(Eval

[GitHub] spark pull request: [SPARK-7225][SQL] CombineLimits optimizer does...

2015-04-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/5770 --- 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-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5799#issuecomment-97667923 I'm going to let @mengxr to comment on the actual algorithm implementation. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29404857 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/ml/FrequentItemsSuite.scala --- @@ -0,0 +1,45 @@ +/* + * Licensed to the Apache Software Foundatio

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29404815 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala --- @@ -1414,4 +1415,37 @@ class DataFrame private[sql]( val jrdd = rdd.map(Evalua

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29404786 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala --- @@ -1414,4 +1415,37 @@ class DataFrame private[sql]( val jrdd = rdd.map(Evalua

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5799#issuecomment-97667662 [Test build #31386 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31386/consoleFull) for PR 5799 at commit [`8279d4d`](https://githu

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29404754 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/ml/FrequentItems.scala --- @@ -0,0 +1,124 @@ +/* +* Licensed to the Apache Software Foundation (AS

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5799#issuecomment-97667532 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-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5799#issuecomment-97667521 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 ha

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5799#discussion_r29404717 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala --- @@ -1414,4 +1415,25 @@ class DataFrame private[sql]( val jrdd = rdd.map(Evalua

[GitHub] spark pull request: [SPARK-7120][SPARK-7121] Closure cleaner nesti...

2015-04-29 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/5685#discussion_r29404655 --- Diff: core/src/test/scala/org/apache/spark/util/ClosureCleanerSuite2.scala --- @@ -0,0 +1,562 @@ +/* + * Licensed to the Apache Software Foundati

[GitHub] spark pull request: [SPARK-7242][SQL][MLLIB] Frequent items for Da...

2015-04-29 Thread brkyvz
GitHub user brkyvz opened a pull request: https://github.com/apache/spark/pull/5799 [SPARK-7242][SQL][MLLIB] Frequent items for DataFrames Finding frequent items with possibly false positives, using the algorithm described in `http://www.cs.umd.edu/~samir/498/karp.pdf`. public A

[GitHub] spark pull request: [SPARK-7225][SQL] CombineLimits optimizer does...

2015-04-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5770#issuecomment-97665645 Thanks. Merging in 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 thi

[GitHub] spark pull request: [Build] Enable MiMa checks for SQL

2015-04-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5727#issuecomment-97665454 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 an

[GitHub] spark pull request: [SPARK-7225][SQL] CombineLimits optimizer does...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5770#issuecomment-97665312 [Test build #31377 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31377/consoleFull) for PR 5770 at commit [`c68eaa7`](https://gith

[GitHub] spark pull request: [SPARK-7225][SQL] CombineLimits optimizer does...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5770#issuecomment-97665315 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-7225][SQL] CombineLimits optimizer does...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5770#issuecomment-97665316 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/313

[GitHub] spark pull request: [SPARK-5521] PCA wrapper for easy transform ve...

2015-04-29 Thread catap
Github user catap commented on a diff in the pull request: https://github.com/apache/spark/pull/4304#discussion_r29404295 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/feature/PCA.scala --- @@ -0,0 +1,109 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] spark pull request: [SPARK-7269] [SQL] Incorrect analysis for aggr...

2015-04-29 Thread scwf
Github user scwf commented on a diff in the pull request: https://github.com/apache/spark/pull/5798#discussion_r29403917 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveResolutionSuite.scala --- @@ -81,9 +81,13 @@ class HiveResolutionSuite extends HiveCom

[GitHub] spark pull request: [SPARK-7232] [SQL] Add a Substitution batch fo...

2015-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5776#issuecomment-97663859 [Test build #31385 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31385/consoleFull) for PR 5776 at commit [`553005a`](https://githu

[GitHub] spark pull request: [SPARK-7269] [SQL] Incorrect analysis for aggr...

2015-04-29 Thread scwf
Github user scwf commented on a diff in the pull request: https://github.com/apache/spark/pull/5798#discussion_r29403863 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/package.scala --- @@ -29,12 +29,23 @@ package object analysis { /**

[GitHub] spark pull request: [SPARK-7031][ThriftServer]let thrift server ta...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5609#issuecomment-97663305 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-7232] [SQL] Add a Substitution batch fo...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5776#issuecomment-97663531 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

  1   2   3   4   5   6   7   8   9   10   >