[GitHub] spark pull request: [SPARK-14800][SQL] Dealing with null as a valu...

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

[GitHub] spark pull request: [SPARK-13745][SQL]Support columnar in memory r...

2016-04-22 Thread robbinspg
Github user robbinspg commented on the pull request: https://github.com/apache/spark/pull/12397#issuecomment-213670252 Can we re test this as I think there was a minor change since the test build --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-13745][SQL]Support columnar in memory r...

2016-04-22 Thread robbinspg
Github user robbinspg commented on the pull request: https://github.com/apache/spark/pull/12501#issuecomment-213670198 closing this in favour of other implementation --- 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-13745][SQL]Support columnar in memory r...

2016-04-22 Thread robbinspg
Github user robbinspg closed the pull request at: https://github.com/apache/spark/pull/12501 --- 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-14551][SQL] Reduce number of NameNode c...

2016-04-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12319 --- 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-14866][SQL] Break SQLQuerySuite out int...

2016-04-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12630 --- 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-14551][SQL] Reduce number of NameNode c...

2016-04-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12319#issuecomment-213669972 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

[GitHub] spark pull request: [SPARK-14866][SQL] Break SQLQuerySuite out int...

2016-04-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12630#issuecomment-213669781 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 this

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

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

[GitHub] spark pull request: [SPARK-14863][SQL] Cache TreeNode's hashCode b...

2016-04-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12626 --- 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-14613][ML] Add @Since into the matrix a...

2016-04-22 Thread pravingadakh
Github user pravingadakh commented on the pull request: https://github.com/apache/spark/pull/12416#issuecomment-213668195 @dbtsai I'll update the PR soon, I have been overwhelmed by office work :( --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-14863][SQL] Cache TreeNode's hashCode b...

2016-04-22 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/12626#issuecomment-213668471 thanks, merging to master! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-14863][SQL] Cache TreeNode's hashCode b...

2016-04-22 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/12626#issuecomment-213667937 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

[GitHub] spark pull request: [SPARK-11102] [SQL] Uninformative exception wh...

2016-04-22 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/9490#issuecomment-213668158 ping @zjffdu --- 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-14857] [SQL] Table/Database Name Valida...

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

[GitHub] spark pull request: [SPARK-14856] [SQL] returning batch correctly

2016-04-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12619 --- 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-14856] [SQL] returning batch correctly

2016-04-22 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/12619#issuecomment-213667183 Merging this 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-14856] [SQL] returning batch correctly

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

[GitHub] spark pull request: [SPARK-14856] [SQL] returning batch correctly

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12619#issuecomment-213666883 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-14856] [SQL] returning batch correctly

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

[GitHub] spark pull request: [SPARK-14866][SQL] Break SQLQuerySuite out int...

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

[GitHub] spark pull request: [SPARK-14866][SQL] Break SQLQuerySuite out int...

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12630#issuecomment-213666751 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-14866][SQL] Break SQLQuerySuite out int...

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

[GitHub] spark pull request: [SPARK-14867][BUILD] Make `build/mvn` to use t...

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

[GitHub] spark pull request: [SPARK-14867][BUILD] Make `build/mvn` to use t...

2016-04-22 Thread dongjoon-hyun
GitHub user dongjoon-hyun opened a pull request: https://github.com/apache/spark/pull/12631 [SPARK-14867][BUILD] Make `build/mvn` to use the downloaded maven if it exists. ## What changes were proposed in this pull request? Currently, `build/mvn` provides a convenient

[GitHub] spark pull request: [SPARK-14830][SQL] Add RemoveRepetitionFromGro...

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

[GitHub] spark pull request: [SPARK-14830][SQL] Add RemoveRepetitionFromGro...

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12590#issuecomment-213666547 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-14830][SQL] Add RemoveRepetitionFromGro...

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

[GitHub] spark pull request: [SPARK-14433][PySpark][ML]:PySpark ml Gaussian...

2016-04-22 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/12402#discussion_r60823427 --- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/GaussianMixture.scala --- @@ -104,6 +105,25 @@ class GaussianMixtureModel private[ml] (

[GitHub] spark pull request: [SPARK-14433][PySpark][ML]:PySpark ml Gaussian...

2016-04-22 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/12402#discussion_r60823428 --- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/GaussianMixture.scala --- @@ -104,6 +105,25 @@ class GaussianMixtureModel private[ml] (

[GitHub] spark pull request: [SPARK-14433][PySpark][ML]:PySpark ml Gaussian...

2016-04-22 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/12402#discussion_r60823426 --- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/GaussianMixture.scala --- @@ -104,6 +105,25 @@ class GaussianMixtureModel private[ml] (

[GitHub] spark pull request: [SPARK-6717][ML] Clear shuffle files after che...

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

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

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

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12568#issuecomment-213663862 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-14731][shuffle]Revert SPARK-12130 to ma...

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

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

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

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12568#issuecomment-213663454 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: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread zhengruifeng
Github user zhengruifeng commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-213663449 @srowen I have reviewed all scala files in Graphx and some in SQL. And remove another some unused imports in this PR. --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

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

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

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

[GitHub] spark pull request: [SPARK-7861][ML] PySpark OneVsRest

2016-04-22 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/12124#issuecomment-213663303 I'm working on a simpler fix for now: [https://issues.apache.org/jira/browse/SPARK-14862] --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

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

[GitHub] spark pull request: [SPARK-14459] [SQL] Detect relation partitioni...

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

[GitHub] spark pull request: [SPARK-14480][SQL] Simplify CSV parsing proces...

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12268#issuecomment-213663159 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-14480][SQL] Simplify CSV parsing proces...

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

[GitHub] spark pull request: [SPARK-14480][SQL] Simplify CSV parsing proces...

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

[GitHub] spark pull request: [SPARK-14459] [SQL] Detect relation partitioni...

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

[GitHub] spark pull request: [SPARK-14838][SQL] Implement statistics in Ser...

2016-04-22 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/12599#discussion_r60823141 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/object.scala --- @@ -83,6 +83,28 @@ case class SerializeFromObject(

[GitHub] spark pull request: [SPARK-14525][SQL] Make DataFrameWrite.save wo...

2016-04-22 Thread JustinPihony
Github user JustinPihony commented on the pull request: https://github.com/apache/spark/pull/12601#issuecomment-213662908 @HyukjinKwon I just posted on the JIRA the background of `Properties` and how reasonable it is to assume it can be converted to a `String`. --- If your project

[GitHub] spark pull request: [SPARK-14551][SQL] Reduce number of NameNode c...

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12319#issuecomment-213662852 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-14551][SQL] Reduce number of NameNode c...

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

[GitHub] spark pull request: [SPARK-14551][SQL] Reduce number of NameNode c...

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

[GitHub] spark pull request: [SPARK-14551][SQL] Reduce number of NameNode c...

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

[GitHub] spark pull request: [SPARK-14800][SQL] Dealing with null as a valu...

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

[GitHub] spark pull request: [SPARK-6717][ML] Clear shuffle files after che...

2016-04-22 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/11919#discussion_r60822985 --- Diff: mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala --- @@ -1306,4 +1306,33 @@ object ALS extends DefaultParamsReadable[ALS] with

[GitHub] spark pull request: [SPARK-8398][CORE] Hadoop input/output format ...

2016-04-22 Thread koertkuipers
Github user koertkuipers commented on the pull request: https://github.com/apache/spark/pull/6848#issuecomment-213661475 @holdenk ok i tried to make it look all pretty --- 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-14856] [SQL] returning batch correctly

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

[GitHub] spark pull request: [SPARK-14654][CORE][WIP] New accumulator API

2016-04-22 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/12612#discussion_r60822740 --- Diff: core/src/main/scala/org/apache/spark/NewAccumulator.scala --- @@ -0,0 +1,299 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] spark pull request: [SPARK-14800][SQL] Dealing with null as a valu...

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12629#issuecomment-213661096 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-14800][SQL] Dealing with null as a valu...

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

[GitHub] spark pull request: [SPARK-14800][SQL] Dealing with null as a valu...

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

[GitHub] spark pull request: [SPARK-14856] [SQL] returning batch correctly

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

[GitHub] spark pull request: [SPARK-14856] [SQL] returning batch correctly

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

[GitHub] spark pull request: [SPARK-14856] [SQL] returning batch correctly

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12619#issuecomment-213660940 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-14856] [SQL] returning batch correctly

2016-04-22 Thread sameeragarwal
Github user sameeragarwal commented on a diff in the pull request: https://github.com/apache/spark/pull/12619#discussion_r60822682 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRelation.scala --- @@ -308,8 +304,11 @@ private[sql]

[GitHub] spark pull request: [SPARK-14856] [SQL] returning batch correctly

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

[GitHub] spark pull request: [SPARK-14830][SQL] Add RemoveRepetitionFromGro...

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

[GitHub] spark pull request: [SPARK-14866][SQL] Break SQLQuerySuite out int...

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

[GitHub] spark pull request: [SPARK-14830][SQL] Add RemoveRepetitionFromGro...

2016-04-22 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/12590#issuecomment-213660622 Rebased. --- 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-14866][SQL] Break SQLQuerySuite out int...

2016-04-22 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/12630 [SPARK-14866][SQL] Break SQLQuerySuite out into smaller test suites ## What changes were proposed in this pull request? This patch breaks SQLQuerySuite out into smaller test suites. It was a

[GitHub] spark pull request: [SPARK-14842][SQL] Implement view creation in ...

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12615#issuecomment-213660185 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-14842][SQL] Implement view creation in ...

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

[GitHub] spark pull request: [SPARK-14842][SQL] Implement view creation in ...

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

[GitHub] spark pull request: [SPARK-14842][SQL] Implement view creation in ...

2016-04-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12615 --- 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-14856] [SQL] returning batch correctly

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

[GitHub] spark pull request: [SPARK-14856] [SQL] returning batch correctly

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12619#issuecomment-213660053 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-14856] [SQL] returning batch correctly

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

[GitHub] spark pull request: [SPARK-9314] [EC2] add root EBS config options...

2016-04-22 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/7647#issuecomment-213659877 ping @kmaehashi --- 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-14842][SQL] Implement view creation in ...

2016-04-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12615#issuecomment-213659905 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 this

[GitHub] spark pull request: [SPARK-14551][SQL] Reduce number of NameNode c...

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

[GitHub] spark pull request: [SPARK-14551][SQL] Reduce number of NameNode c...

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

[GitHub] spark pull request: [SPARK-8603] [sparkR] In windows, Incorrect fi...

2016-04-22 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/7025#issuecomment-213659754 ping @prakashpc --- 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-8603] [sparkR] In windows, Incorrect fi...

2016-04-22 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/7025#issuecomment-213659474 @JoshRosen I can submit a PR based on this if you think this PR is abandoned. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-14551][SQL] Reduce number of NameNode c...

2016-04-22 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/12319#issuecomment-213659267 LGTM pending Jenkins. --- 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-14551][SQL] Reduce number of NameNode c...

2016-04-22 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/12319#issuecomment-213658964 add to whitelist --- 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-14551][SQL] Reduce number of NameNode c...

2016-04-22 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/12319#issuecomment-213658842 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 this

[GitHub] spark pull request: [SPARK-14856] [SQL] returning batch correctly

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

[GitHub] spark pull request: [SPARK-14480][SQL] Simplify CSV parsing proces...

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

[GitHub] spark pull request: [SPARK-14856] [SQL] returning batch correctly

2016-04-22 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/12619#discussion_r60822250 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRelation.scala --- @@ -308,8 +304,11 @@ private[sql] class

[GitHub] spark pull request: [SPARK-14828][SQL] Start SparkSession in REPL ...

2016-04-22 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/12589#discussion_r60822234 --- Diff: repl/scala-2.10/src/main/scala/org/apache/spark/repl/SparkILoop.scala --- @@ -1026,21 +1025,7 @@ class SparkILoop( }

[GitHub] spark pull request: [SPARK-14480][SQL] Simplify CSV parsing proces...

2016-04-22 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/12268#issuecomment-213658002 @rxin Could you please review 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

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

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

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

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

[GitHub] spark pull request: [SPARK-14800][SQL] Dealing with null as a valu...

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

[GitHub] spark pull request: [SPARK-14800][SQL] Dealing with null as a valu...

2016-04-22 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/12629#issuecomment-213656738 cc @davies @viirya --- 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-14800][SQL] Dealing with null as a valu...

2016-04-22 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/12629 [SPARK-14800][SQL] Dealing with null as a value in options for each internal data source ## What changes were proposed in this pull request?

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

2016-04-22 Thread lianhuiwang
Github user lianhuiwang commented on a diff in the pull request: https://github.com/apache/spark/pull/12568#discussion_r60822093 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala --- @@ -182,7 +182,7 @@ private[spark] class BlockManager( val

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

2016-04-22 Thread lianhuiwang
Github user lianhuiwang commented on the pull request: https://github.com/apache/spark/pull/12568#issuecomment-213656641 @vanzin I have addressed your comments. 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

[GitHub] spark pull request: [SPARK-14731][shuffle]Revert SPARK-12130 to ma...

2016-04-22 Thread lianhuiwang
Github user lianhuiwang commented on a diff in the pull request: https://github.com/apache/spark/pull/12568#discussion_r60821688 --- Diff: common/network-shuffle/src/test/java/org/apache/spark/network/shuffle/ExternalShuffleIntegrationSuite.java --- @@ -184,12 +184,9 @@ public

[GitHub] spark pull request: [SPARK-14863][SQL] Cache TreeNode's hashCode b...

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

  1   2   3   4   5   6   7   8   9   10   >