[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7478#issuecomment-122505596 [Test build #37696 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37696/consoleFull) for PR 7478 at commit [`3a53278`](https://gith

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7478#issuecomment-122505389 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-8007][SQL] Added virtual column support...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7478#issuecomment-122505334 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7478#issuecomment-122504605 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-8007][SQL] Added virtual column support...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7478#issuecomment-122504600 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [Spark-7422][MLLIB] Add argmax to Vector, Spar...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6112#issuecomment-122504545 [Test build #37694 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37694/consoleFull) for PR 6112 at commit [`3e0a939`](https://gith

[GitHub] spark pull request: [SPARK-8443][SQL] Split GenerateMutableProject...

2015-07-17 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7076#discussion_r34945166 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateMutableProjection.scala --- @@ -45,7 +45,37 @@ object GenerateMu

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread JDrit
Github user JDrit commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34945159 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveTableScanSuite.scala --- @@ -90,4 +90,11 @@ class HiveTableScanSuite extends HiveComp

[GitHub] spark pull request: [SPARK-8443][SQL] Split GenerateMutableProject...

2015-07-17 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7076#discussion_r34945162 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateMutableProjection.scala --- @@ -45,7 +45,37 @@ object GenerateMu

[GitHub] spark pull request: [Spark-7422][MLLIB] Add argmax to Vector, Spar...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6112#issuecomment-122504419 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-7422][MLLIB] Add argmax to Vector, Spar...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6112#issuecomment-122504417 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8271][SQL]string function: soundex

2015-07-17 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/7115#discussion_r34945097 --- Diff: unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java --- @@ -515,4 +521,91 @@ public int hashCode() { } return result

[GitHub] spark pull request: [SPARK-8271][SQL]string function: soundex

2015-07-17 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/7115#discussion_r34945058 --- Diff: unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java --- @@ -515,4 +521,91 @@ public int hashCode() { } return result

[GitHub] spark pull request: [SPARK-8240][SQL] string function: concat

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7486#issuecomment-122493336 [Test build #37693 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37693/consoleFull) for PR 7486 at commit [`66bfbee`](https://gith

[GitHub] spark pull request: [SPARK-8240][SQL] string function: concat

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7486#issuecomment-122491894 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8240][SQL] string function: concat

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7486#issuecomment-122491967 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: [MLlib][Doc] Seed fix in mllib naive bayes exa...

2015-07-17 Thread mupakoz
Github user mupakoz commented on the pull request: https://github.com/apache/spark/pull/7477#issuecomment-122490168 Of course, no problem :). I'll prepare the commit later today. --- 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-8240][SQL] string function: concat

2015-07-17 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7486#discussion_r34944828 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala --- @@ -208,169 +208,6 @@ class DataFrameFunctionsSuite extends QueryTest {

[GitHub] spark pull request: [SPARK-8240][SQL] string function: concat

2015-07-17 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/7486 [SPARK-8240][SQL] string function: concat You can merge this pull request into a Git repository by running: $ git pull https://github.com/rxin/spark concat Alternatively you can review and apply

[GitHub] spark pull request: [SPARK-8797] [SPARK-9146] [WIP] Fix comparison...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7194#issuecomment-122489119 [Test build #37692 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37692/consoleFull) for PR 7194 at commit [`3998ef2`](https://gith

[GitHub] spark pull request: [SPARK-8797] [SPARK-9146] [WIP] Fix comparison...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7194#issuecomment-122489096 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-8797] [SPARK-9146] [WIP] Fix comparison...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7194#issuecomment-122489093 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8797] [SPARK-9146] [WIP] Fix comparison...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7194#issuecomment-122489019 [Test build #37691 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37691/consoleFull) for PR 7194 at commit [`a2ba2e7`](https://gith

[GitHub] spark pull request: [SPARK-8797] [SPARK-9146] [WIP] Fix comparison...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7194#issuecomment-122488989 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8797] [SPARK-9146] [WIP] Fix comparison...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7194#issuecomment-122488991 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-8443][SQL] Split GenerateMutableProject...

2015-07-17 Thread saurfang
Github user saurfang commented on the pull request: https://github.com/apache/spark/pull/7076#issuecomment-122485318 I have pushed a new commit that if only one block is generated, then projections will be inlined as before. Can you please review? Or do you prefer I doing the other ap

[GitHub] spark pull request: [SPARK-9128][Core] Get outerclasses and object...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7459#issuecomment-122485381 [Test build #37690 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37690/consoleFull) for PR 7459 at commit [`2ec5ce1`](https://gith

[GitHub] spark pull request: [SPARK-9128][Core] Get outerclasses and object...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7459#issuecomment-122485082 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-9128][Core] Get outerclasses and object...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7459#issuecomment-122485080 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8443][SQL] Split GenerateMutableProject...

2015-07-17 Thread saurfang
Github user saurfang commented on a diff in the pull request: https://github.com/apache/spark/pull/7076#discussion_r34944623 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CodeGenerationSuite.scala --- @@ -42,4 +42,8 @@ class CodeGenerationSuite ex

[GitHub] spark pull request: [SPARK-9128][Core] Get outerclasses and object...

2015-07-17 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/7459#issuecomment-122485016 @dragos @andrewor14 thanks for suggestions. I updated this and removed the two methods. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-9114] [SQL] [PySpark] convert returned ...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7450#issuecomment-122484843 [Test build #1098 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1098/console) for PR 7450 at commit [`e80bf9f`](https://github

[GitHub] spark pull request: [SPARK-9142][SQL] Removing unnecessary self ty...

2015-07-17 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/7479#discussion_r34944515 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala --- @@ -25,8 +25,7 @@ import org.apache.spark.sql.catal

[GitHub] spark pull request: [SPARK-6776] [SQL] [WIP] More Parquet compatib...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7472#issuecomment-122483794 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-6776] [SQL] [WIP] More Parquet compatib...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7472#issuecomment-122483776 [Test build #37687 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37687/console) for PR 7472 at commit [`b1be549`](https://github.

[GitHub] spark pull request: [SPARK-6126][SQL] added coverage for UDTs in J...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7416#discussion_r34944353 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/UserDefinedTypeSuite.scala --- @@ -34,6 +34,8 @@ private[sql] class MyDenseVector(val data: Array[Double]

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/7478#issuecomment-122483507 @JDrit Just finished my review. Seems we do not need to make any change in `LogicalPlan.scala`. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-9142][SQL] Removing unnecessary self ty...

2015-07-17 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/7479#discussion_r34944321 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala --- @@ -25,8 +25,7 @@ import org.apache.spark.sql.catal

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34944326 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/VirtualColumnSuite.scala --- @@ -0,0 +1,37 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34944322 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/VirtualColumnSuite.scala --- @@ -0,0 +1,37 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34944318 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveTableScanSuite.scala --- @@ -90,4 +90,11 @@ class HiveTableScanSuite extends HiveComp

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34944319 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveTableScanSuite.scala --- @@ -90,4 +90,11 @@ class HiveTableScanSuite extends HiveComp

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34944312 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/ColumnExpressionSuite.scala --- @@ -17,6 +17,7 @@ package org.apache.spark.sql +impo

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34944314 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala --- @@ -37,6 +37,7 @@ import org.apache.spark.{HashPartitioner, SparkEnv}

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34944311 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala --- @@ -1018,6 +1022,20 @@ class SQLContext(@transient val sparkContext: SparkContext)

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34944231 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala --- @@ -212,6 +212,25 @@ abstract class LogicalPlan extends

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34944170 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala --- @@ -212,6 +212,25 @@ abstract class LogicalPlan extends

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34944166 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala --- @@ -235,10 +254,13 @@ abstract class LogicalPlan extend

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34944153 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala --- @@ -235,10 +254,13 @@ abstract class LogicalPlan extend

[GitHub] spark pull request: [SPARK-8881] Fix algorithm for scheduling exec...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7274#issuecomment-122480553 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-8881] Fix algorithm for scheduling exec...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7274#issuecomment-122480515 [Test build #37684 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37684/console) for PR 7274 at commit [`f279cdf`](https://github.

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7478#issuecomment-122480369 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-8007][SQL] Added virtual column support...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7478#issuecomment-122480356 [Test build #37686 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37686/console) for PR 7478 at commit [`ae7e747`](https://github.

[GitHub] spark pull request: [SPARK-8998][MLlib] Collect enough frequent pr...

2015-07-17 Thread zhangjiajin
Github user zhangjiajin commented on a diff in the pull request: https://github.com/apache/spark/pull/7412#discussion_r34943941 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala --- @@ -82,20 +86,69 @@ class PrefixSpan private ( logWarning("Input

[GitHub] spark pull request: [SPARK-8998][MLlib] Collect enough frequent pr...

2015-07-17 Thread zhangjiajin
Github user zhangjiajin commented on a diff in the pull request: https://github.com/apache/spark/pull/7412#discussion_r34943942 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala --- @@ -82,20 +86,69 @@ class PrefixSpan private ( logWarning("Input

[GitHub] spark pull request: [SPARK-8924] [MLLIB, DOCUMENTATION] Added @sin...

2015-07-17 Thread manishamde
Github user manishamde commented on the pull request: https://github.com/apache/spark/pull/7380#issuecomment-122480109 @mengxr Sure. I can take a look. How do we handle API modifications? Any change moves the tag to the newest version? --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-8246] [SQL] Implement get_json_object

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7485#issuecomment-122479950 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-8246] [SQL] Implement get_json_object

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7485#issuecomment-122479937 [Test build #37685 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37685/console) for PR 7485 at commit [`6349bbc`](https://github.

[GitHub] spark pull request: [SPARK-8364][SPARKR] Add crosstab to SparkR Da...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7318#issuecomment-122479710 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-8364][SPARKR] Add crosstab to SparkR Da...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7318#issuecomment-122479699 [Test build #37688 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37688/console) for PR 7318 at commit [`d75e894`](https://github.

[GitHub] spark pull request: [SPARK-9099] [EC2] Add more ports to security ...

2015-07-17 Thread serialx
Github user serialx commented on the pull request: https://github.com/apache/spark/pull/7443#issuecomment-122479463 @JoshRosen @andrewor14 The ports added are from options flags '--authorized-address' which should be set to a secure ip origin. So, it's not opening to the world. I thin

[GitHub] spark pull request: [SPARK-9114] [SQL] [PySpark] convert returned ...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7450#issuecomment-122479420 [Test build #1098 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1098/consoleFull) for PR 7450 at commit [`e80bf9f`](https://git

[GitHub] spark pull request: [SPARK-8797] [SPARK-9146] [WIP] Fix comparison...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7194#issuecomment-122479056 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 does n

[GitHub] spark pull request: [SPARK-8797] [SPARK-9146] [WIP] Fix comparison...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7194#issuecomment-122479053 [Test build #37682 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37682/console) for PR 7194 at commit [`630ebc5`](https://github.

[GitHub] spark pull request: [SPARK-8199][SPARK-8184][SPARK-8183][SPARK-818...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6981#issuecomment-122479020 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-8199][SPARK-8184][SPARK-8183][SPARK-818...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6981#issuecomment-122479010 [Test build #37680 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37680/console) for PR 6981 at commit [`4afc09c`](https://github.

[GitHub] spark pull request: [SPARK-8797] [SPARK-9146] [WIP] Fix comparison...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7194#issuecomment-122478504 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-8924] [MLLIB, DOCUMENTATION] Added @sin...

2015-07-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/7380#issuecomment-122477450 @manishamde @chouqin Could you make a pass? --- 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-9143] [SQL] Add planner rule for automa...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7482#issuecomment-122477415 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: [MLlib][Doc] Seed fix in mllib naive bayes exa...

2015-07-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/7477#issuecomment-122477418 @mupakoz I got the same result as @srowen . I don't think changing the seed is the solution. Instead, we should add more rows to `sample_naive_bayes_data.txt`. Could you

[GitHub] spark pull request: [SPARK-9143] [SQL] Add planner rule for automa...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7482#issuecomment-122477395 [Test build #37681 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37681/console) for PR 7482 at commit [`5220cce`](https://github.

[GitHub] spark pull request: [SPARK-1855] Local checkpointing

2015-07-17 Thread witgo
Github user witgo commented on the pull request: https://github.com/apache/spark/pull/7279#issuecomment-122477288 This is a very good performance for ML.I will test this feature on [cloudml/zen](https://github.com/cloudml/zen) --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-8364][SPARKR] Add crosstab to SparkR Da...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7318#issuecomment-122477226 [Test build #37688 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37688/consoleFull) for PR 7318 at commit [`d75e894`](https://gith

[GitHub] spark pull request: [Spark-7879][MLlib] KMeans API for spark.ml Pi...

2015-07-17 Thread yu-iskw
Github user yu-iskw commented on the pull request: https://github.com/apache/spark/pull/6756#issuecomment-122477206 Thank you for merging it and your continuous support! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-9118][ML]: Implement IntArrayParam in m...

2015-07-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/7481#issuecomment-122477162 LGTM. Merged into master. Thanks! (It would be better to implement this when we really need it.) --- If your project is set up for it, you can reply to this email and ha

[GitHub] spark pull request: [SPARK-9118][ML]: Implement IntArrayParam in m...

2015-07-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7481 --- 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-9144] Remove DAGScheduler.runLocallyWit...

2015-07-17 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7484#issuecomment-122477150 I think the idea is to run stuff like take much faster. Why do we want to remove this? --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] spark pull request: [SPARK-8364][SPARKR] Add crosstab to SparkR Da...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7318#issuecomment-122477110 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8797] [SPARK-9146] [WIP] Fix comparison...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7194#issuecomment-122477117 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-8364][SPARKR] Add crosstab to SparkR Da...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7318#issuecomment-122477116 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-8797] [SPARK-9146] [WIP] Fix comparison...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7194#issuecomment-122477114 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8364][SPARKR] Add crosstab to SparkR Da...

2015-07-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/7318#issuecomment-122477096 @shivaram This should be ready for review after Jenkins. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] spark pull request: [SPARK-8364][SPARKR] Add crosstab to SparkR Da...

2015-07-17 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/7318#discussion_r34943565 --- Diff: R/pkg/inst/tests/test_sparkSQL.R --- @@ -949,6 +949,17 @@ test_that("fillna() on a DataFrame", { expect_identical(expected, actual) })

[GitHub] spark pull request: [SPARK-6776] [SQL] [WIP] More Parquet compatib...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7472#issuecomment-122476714 [Test build #37687 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37687/consoleFull) for PR 7472 at commit [`b1be549`](https://gith

[GitHub] spark pull request: [SPARK-6776] [SQL] [WIP] More Parquet compatib...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7472#issuecomment-122476591 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-6776] [SQL] [WIP] More Parquet compatib...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7472#issuecomment-122476594 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-6776] [SQL] [WIP] More Parquet compatib...

2015-07-17 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/7472#issuecomment-122476586 Added a wrong entry to `.gitignore`, generated `Suit.java` wasn't committed, and caused the last build error. --- If your project is set up for it, you can reply to t

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7478#discussion_r34943299 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala --- @@ -235,10 +254,13 @@ abstract class LogicalPlan extend

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7478#issuecomment-122473476 [Test build #37686 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37686/consoleFull) for PR 7478 at commit [`ae7e747`](https://gith

[GitHub] spark pull request: [SPARK-8246] [SQL] Implement get_json_object

2015-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7485#issuecomment-122472989 [Test build #37685 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37685/consoleFull) for PR 7485 at commit [`6349bbc`](https://gith

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7478#issuecomment-122472556 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8246] [SQL] Implement get_json_object

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7485#issuecomment-122472591 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-8007][SQL] Added virtual column support...

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7478#issuecomment-122472590 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-8246] [SQL] Implement get_json_object

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7485#issuecomment-122472548 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8246] [SQL] Implement get_json_object

2015-07-17 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7485#issuecomment-12247 Jenkins, test this please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have thi

[GitHub] spark pull request: [SPARK-8007][SQL] Added virtual column support...

2015-07-17 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/7478#issuecomment-122472235 ok to test --- 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 enabl

[GitHub] spark pull request: [SPARK-8246] [SQL] Implement get_json_object

2015-07-17 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7485#issuecomment-122472184 Yup you can add it to FunctionRegistry and it will be available in the parser automatically after that. --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-8246] [SQL] Implement get_json_object

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7485#issuecomment-122469640 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your pr

[GitHub] spark pull request: [SPARK-9136][SQL] fix several bugs in DateTime...

2015-07-17 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/7473#discussion_r34943083 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala --- @@ -320,16 +320,17 @@ object DateTimeUtils { Ca

[GitHub] spark pull request: [SPARK-8246] [SQL] Implement get_json_object

2015-07-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7485#issuecomment-122469154 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your pr

  1   2   3   4   5   6   7   8   9   10   >