[GitHub] spark pull request: [SPARK-13296][SQL] Move UserDefinedFunction in...

2016-02-12 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/11181 [SPARK-13296][SQL] Move UserDefinedFunction into sql.expressions. This is more consistent with how we structure the packages for window functions and UDAFs. I will have a follow-up pull request to

[GitHub] spark pull request: [SPARK-13296][SQL] Move UserDefinedFunction in...

2016-02-12 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11181#issuecomment-183245123 mima will probably fail --- 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-12962] [SQL] [PySpark] PySpark support ...

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

[GitHub] spark pull request: [SPARK-13282][SQL] LogicalPlan toSql should ju...

2016-02-12 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11171#discussion_r52714801 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/SQLBuilderTest.scala --- @@ -50,10 +50,8 @@ abstract class SQLBuilderTest extends QueryTest

[GitHub] spark pull request: [SPARK-13282][SQL] LogicalPlan toSql should ju...

2016-02-12 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11171#discussion_r52714791 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/SQLBuilderTest.scala --- @@ -50,10 +50,8 @@ abstract class SQLBuilderTest extends QueryTest with

[GitHub] spark pull request: [SPARK-13282][SQL] LogicalPlan toSql should ju...

2016-02-12 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11171#discussion_r52714855 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/SQLBuilderTest.scala --- @@ -50,10 +50,8 @@ abstract class SQLBuilderTest extends QueryTest with

[GitHub] spark pull request: [SPARK-10521][SQL] Utilize Docker for test DB2...

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

[GitHub] spark pull request: [SPARK-13282][SQL] LogicalPlan toSql should ju...

2016-02-12 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11171#issuecomment-183226591 Updated. cc @gatorsmile unfortunately you will have to rebase your pull requests, although it should be easy to do. --- If your project is set up for it, you

[GitHub] spark pull request: [Docs] Update cache()'s storage level to be co...

2016-02-12 Thread wjur
Github user wjur commented on the pull request: https://github.com/apache/spark/pull/11172#issuecomment-183226607 @mateiz, I'm not sure who is responsible for the docs, but it it seems that you are the last one who edited the affected lines. I would be grateful if you could take a

[GitHub] spark pull request: SPARK-12729 PhantomReferences to replace Final...

2016-02-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11140#issuecomment-183227808 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-12729 PhantomReferences to replace Final...

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

[GitHub] spark pull request: [SPARK-13296][SQL] Move UserDefinedFunction in...

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

[GitHub] spark pull request: [SPARK-13282][SQL] LogicalPlan toSql should ju...

2016-02-12 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11171#discussion_r52714511 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala --- @@ -37,157 +39,137 @@ import

[GitHub] spark pull request: [SPARK-13282][SQL] LogicalPlan toSql should ju...

2016-02-12 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11171#discussion_r52714530 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala --- @@ -37,157 +39,137 @@ import

[GitHub] spark pull request: [SPARK-13282][SQL] LogicalPlan toSql should ju...

2016-02-12 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11171#discussion_r52714504 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala --- @@ -37,157 +39,137 @@ import

[GitHub] spark pull request: [SPARK-13282][SQL] LogicalPlan toSql should ju...

2016-02-12 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11171#discussion_r52714696 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/SQLBuilderTest.scala --- @@ -50,10 +50,8 @@ abstract class SQLBuilderTest extends QueryTest

[GitHub] spark pull request: [SPARK-10780][ML][WIP] Add initial model to km...

2016-02-12 Thread dbtsai
Github user dbtsai commented on the pull request: https://github.com/apache/spark/pull/9#issuecomment-183248930 @yinxusen I'll be away for Spark summit east. Gonna work on this again when I'm back. Thanks. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-10780][ML][WIP] Add initial model to km...

2016-02-12 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/9#issuecomment-183249963 Never mind, take your time. 2016年2月12日星期五,DB Tsai 写道: > @yinxusen I'll be

[GitHub] spark pull request: [SPARK-13282][SQL] LogicalPlan toSql should ju...

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

[GitHub] spark pull request: [SPARK-10521][SQL] Utilize Docker for test DB2...

2016-02-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9893#issuecomment-183261650 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-10521][SQL] Utilize Docker for test DB2...

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

[GitHub] spark pull request: SPARK-12729 PhantomReferences to replace Final...

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

[GitHub] spark pull request: [SPARK-12705] [SQL] push missing attributes fo...

2016-02-12 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11153#discussion_r52717321 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -572,98 +572,64 @@ class Analyzer( // Skip

[GitHub] spark pull request: [SPARK-12705] [SQL] push missing attributes fo...

2016-02-12 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11153#issuecomment-183238642 LGTM except one comment --- 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-13296][SQL] Move UserDefinedFunction in...

2016-02-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11181#issuecomment-183251557 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-13296][SQL] Move UserDefinedFunction in...

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

[GitHub] spark pull request: [SPARK-12363][Mllib] Remove setRun and fix Pow...

2016-02-12 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/10539#issuecomment-183254208 @viirya I sent you an update at https://github.com/viirya/spark-1/pull/2. Could you review and merge it if it looks good to you? That should fix the PIC implementation.

[GitHub] spark pull request: [SPARK-10521][SQL] Utilize Docker for test DB2...

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

[GitHub] spark pull request: [SPARK-12962] [SQL] [PySpark] PySpark support ...

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

[GitHub] spark pull request: [SPARK-12962] [SQL] [PySpark] PySpark support ...

2016-02-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10876#issuecomment-183263040 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-12962] [SQL] [PySpark] PySpark support ...

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

[GitHub] spark pull request: [SPARK-7889] [CORE] HistoryServer to refresh c...

2016-02-12 Thread steveloughran
Github user steveloughran closed the pull request at: https://github.com/apache/spark/pull/6935 --- 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

[GitHub] spark pull request: [SPARK-13282][SQL] LogicalPlan toSql should ju...

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

[GitHub] spark pull request: [SPARK-12705] [SQL] push missing attributes fo...

2016-02-12 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11153#issuecomment-183228279 cc @cloud-fan for review too --- 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-12503][SPARK-12505] Limit pushdown in U...

2016-02-12 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/11121#discussion_r52715983 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala --- @@ -91,6 +91,14 @@ abstract class LogicalPlan

[GitHub] spark pull request: [SPARK-12503][SPARK-12505] Limit pushdown in U...

2016-02-12 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/11121#issuecomment-183244562 LGTM except one comment --- 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-13296][SQL] Move UserDefinedFunction in...

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

[GitHub] spark pull request: [SPARK-12974] [ML] [PySpark] Add Python API fo...

2016-02-12 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/10889#issuecomment-183255063 LGTM. Merged into 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 project does

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

2016-02-12 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/10152#discussion_r52722338 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/feature/Word2Vec.scala --- @@ -289,24 +301,20 @@ class Word2Vec extends Serializable with Logging {

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

2016-02-12 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/10152#discussion_r52722467 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/feature/Word2Vec.scala --- @@ -289,24 +301,20 @@ class Word2Vec extends Serializable with Logging {

[GitHub] spark pull request: [SPARK-13296][SQL] Move UserDefinedFunction in...

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

[GitHub] spark pull request: [SPARK-13296][SQL] Move UserDefinedFunction in...

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

[GitHub] spark pull request: [SPARK-13296][SQL] Move UserDefinedFunction in...

2016-02-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11181#issuecomment-183262340 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-12974] [ML] [PySpark] Add Python API fo...

2016-02-12 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/10889#issuecomment-183240342 @mengxr Actually, I have already updated this PR after #10216 get merged. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-13263] [SQL] SQL Generation Support for...

2016-02-12 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11148#discussion_r52715605 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala --- @@ -119,6 +120,18 @@ class SQLBuilder(logicalPlan: LogicalPlan, sqlContext:

[GitHub] spark pull request: [Documentation] Added pygments.rb dependancy

2016-02-12 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/11180#issuecomment-183255397 @amitdev The PR also contains ml-guide changes. Could you remove that commit from your branch? --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-12974] [ML] [PySpark] Add Python API fo...

2016-02-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10889 --- 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-13154][PYTHON] Add linting for pydocs

2016-02-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11109 --- 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-13154][PYTHON] Add linting for pydocs

2016-02-12 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/11109#issuecomment-183261151 LGTM. Merged into 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 project does

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

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

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

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

[GitHub] spark pull request: [SPARK-12746][ML] ArrayType(_, true) should al...

2016-02-12 Thread Earthson
Github user Earthson commented on the pull request: https://github.com/apache/spark/pull/10697#issuecomment-183275317 @mengxr ok, I'll have a look:) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] spark pull request: [SPARK-13172] [CORE] [SQL] Stop using RichExce...

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

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

2016-02-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10411#discussion_r52728479 --- Diff: docs/ml-collaborative-filtering.md --- @@ -0,0 +1,148 @@ +--- +layout: global +title: Collaborative Filtering - spark.ml

[GitHub] spark pull request: Added missing utility method

2016-02-12 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11173#issuecomment-183266153 I don't think we want yet another huge overload. Can't you pass an empty map of offsets? if not, let's just support that. @koeninger --- If your project is set up for

[GitHub] spark pull request: [SPARK-13172] [CORE] [SQL] Stop using RichExce...

2016-02-12 Thread srowen
GitHub user srowen opened a pull request: https://github.com/apache/spark/pull/11182 [SPARK-13172] [CORE] [SQL] Stop using RichException.getStackTrace it is deprecated Replace `getStackTraceString` with `Utils.exceptionString` You can merge this pull request into a Git repository

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

2016-02-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10411#discussion_r52728448 --- Diff: docs/ml-collaborative-filtering.md --- @@ -0,0 +1,148 @@ +--- +layout: global +title: Collaborative Filtering - spark.ml

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

2016-02-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10411#discussion_r52728349 --- Diff: docs/ml-collaborative-filtering.md --- @@ -0,0 +1,148 @@ +--- +layout: global +title: Collaborative Filtering - spark.ml

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

2016-02-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10411#discussion_r52728386 --- Diff: docs/ml-collaborative-filtering.md --- @@ -0,0 +1,148 @@ +--- +layout: global +title: Collaborative Filtering - spark.ml

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

2016-02-12 Thread BenFradet
Github user BenFradet commented on a diff in the pull request: https://github.com/apache/spark/pull/10411#discussion_r52739112 --- Diff: docs/ml-collaborative-filtering.md --- @@ -0,0 +1,148 @@ +--- +layout: global +title: Collaborative Filtering - spark.ml

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

2016-02-12 Thread BenFradet
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/10411#issuecomment-183324222 @srowen thanks for the review, will make the necessary changes. --- 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-6761][SQL] Approximate quantile for Dat...

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

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

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

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

2016-02-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10411#discussion_r52728138 --- Diff: docs/ml-collaborative-filtering.md --- @@ -0,0 +1,148 @@ +--- +layout: global +title: Collaborative Filtering - spark.ml

[GitHub] spark pull request: [SPARK-13263] [SQL] SQL Generation Support for...

2016-02-12 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11148#discussion_r52728653 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/SQLBuilder.scala --- @@ -119,6 +120,18 @@ class SQLBuilder(logicalPlan: LogicalPlan,

[GitHub] spark pull request: [SPARK-13172] [CORE] [SQL] Stop using RichExce...

2016-02-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11182#issuecomment-183314618 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-6761][SQL] Approximate quantile for Dat...

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

[GitHub] spark pull request: [SPARK-13172] [CORE] [SQL] Stop using RichExce...

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

[GitHub] spark pull request: [SPARK-12363][Mllib] Remove setRun and fix Pow...

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

[GitHub] spark pull request: [SPARK-12363][Mllib] Remove setRun and fix Pow...

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

[GitHub] spark pull request: [SPARK-12363][Mllib] Remove setRun and fix Pow...

2016-02-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10539#issuecomment-183315960 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-6761][SQL] Approximate quantile for Dat...

2016-02-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6042#issuecomment-183265619 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-12962] [SQL] [PySpark] PySpark support ...

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

[GitHub] spark pull request: [SPARK-12962] [SQL] [PySpark] PySpark support ...

2016-02-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10876#issuecomment-183277864 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-13263] [SQL] SQL Generation Support for...

2016-02-12 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/11148#discussion_r52729051 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/LogicalPlanToSQLSuite.scala --- @@ -145,6 +145,16 @@ class LogicalPlanToSQLSuite extends

[GitHub] spark pull request: [SPARK-12962] [SQL] [PySpark] PySpark support ...

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

[GitHub] spark pull request: [SPARK-13172] [CORE] [SQL] Stop using RichExce...

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

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

2016-02-12 Thread BenFradet
Github user BenFradet commented on a diff in the pull request: https://github.com/apache/spark/pull/10411#discussion_r52738715 --- Diff: examples/src/main/scala/org/apache/spark/examples/ml/ALSExample.scala --- @@ -0,0 +1,82 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-13117][Web UI] WebUI should use the loc...

2016-02-12 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11133#issuecomment-183266743 @devaraj-kavali this might be a legitimate failure. Does it pass for you? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-13186][Streaming]Migrate away from Sync...

2016-02-12 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11104#issuecomment-183267563 @huaxingao can you rebase 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 project does not

[GitHub] spark pull request: [SPARK-12962] [SQL] [PySpark] PySpark support ...

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

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

2016-02-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10411#discussion_r52727772 --- Diff: examples/src/main/scala/org/apache/spark/examples/ml/ALSExample.scala --- @@ -0,0 +1,82 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [Docs] Update cache()'s storage level to be co...

2016-02-12 Thread wjur
Github user wjur closed the pull request at: https://github.com/apache/spark/pull/11172 --- 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-12247] [ML] [DOC] Documentation for spa...

2016-02-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10411#discussion_r52727882 --- Diff: examples/src/main/java/org/apache/spark/examples/ml/JavaALSExample.java --- @@ -0,0 +1,131 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-12247] [ML] [DOC] Documentation for spa...

2016-02-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10411#discussion_r52727907 --- Diff: examples/src/main/java/org/apache/spark/examples/ml/JavaALSExample.java --- @@ -0,0 +1,131 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [Docs] Update cache()'s storage level to be co...

2016-02-12 Thread wjur
Github user wjur commented on the pull request: https://github.com/apache/spark/pull/11172#issuecomment-183272710 That's confusing... @srowen, you're correct. RDDs are not DataFrames. I'm really sorry for taking your time. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-12363][Mllib] Remove setRun and fix Pow...

2016-02-12 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10539#issuecomment-183309759 @mengxr The update looks good, I've merged it. Thanks. As you modified the test cases in `PowerIterationClusteringSuite`, a minor question is the original test cases in

[GitHub] spark pull request: [SPARK-12363][Mllib] Remove setRun and fix Pow...

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

[GitHub] spark pull request: [Docs] Update cache()'s storage level to be co...

2016-02-12 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11172#issuecomment-183265666 I don't think that's correct: ``` /** Persist this RDD with the default storage level (`MEMORY_ONLY`). */ def persist(): this.type =

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6042#issuecomment-183315146 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-6761][SQL] Approximate quantile for Dat...

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

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

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

[GitHub] spark pull request: Added missing utility method

2016-02-12 Thread koeninger
Github user koeninger commented on the pull request: https://github.com/apache/spark/pull/11173#issuecomment-183330790 Kafkacluster methods are public now, you can do this yourself. I think its fine to just have two overloads, the easy mode and the flexible mode. --- If your

[GitHub] spark pull request: [SPARK-13267] [Web UI] document the ?param arg...

2016-02-12 Thread steveloughran
Github user steveloughran commented on the pull request: https://github.com/apache/spark/pull/11152#issuecomment-183331075 EndpointMeaning /applications A list of all applications ?status=[completed|running] list only

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6042#issuecomment-183356232 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-11701][SPARK-13054] dynamic allocation ...

2016-02-12 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/10951#issuecomment-183358808 Actually I see now a couple of the failures are due to throwing the Exception for commit denied so I'll look at those tests closer --- If your project is set up for

[GitHub] spark pull request: [SPARK-11373] [CORE] Add metrics to the Histor...

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

[GitHub] spark pull request: [SPARK-11373] [CORE] Add metrics to the Histor...

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

[GitHub] spark pull request: [SPARK-11373] [CORE] Add metrics to the Histor...

2016-02-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9571#issuecomment-183332469 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: [Documentation] Added pygments.rb dependancy

2016-02-12 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11180#issuecomment-183351400 @amitdev can you connect this to https://issues.apache.org/jira/browse/SPARK-13300 in the title? it might be the same thing? It looks like lots of the examples

  1   2   3   4   >