[GitHub] spark pull request: [WIP][SPARK-6320][SQL] Move planLater method i...

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

[GitHub] spark pull request: [SPARK-15616] [SQL] Metastore relation should ...

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

[GitHub] spark pull request: [SPARK-15616] [SQL] Metastore relation should ...

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

[GitHub] spark pull request: [SPARK-15616] [SQL] Metastore relation should ...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13373#issuecomment-91691 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-15616] [SQL] Metastore relation should ...

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

[GitHub] spark pull request: [SPARK-15616] [SQL] Metastore relation should ...

2016-05-27 Thread lianhuiwang
GitHub user lianhuiwang opened a pull request: https://github.com/apache/spark/pull/13373 [SPARK-15616] [SQL] Metastore relation should fallback to HDFS size of partitions that are involved in Query if statistics are not available. ## What changes were proposed in this pull

[GitHub] spark pull request: [SPARK-15585][SQL] Fix NULL handling along wit...

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

[GitHub] spark pull request: [SPARK-15585][SQL] Fix NULL handling along wit...

2016-05-27 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/13372#issuecomment-91079 @rxin Could you check this to satisfy your suggestion? If no problem, I'll also set default values in json options in a similar way. --- If your project is set up for

[GitHub] spark pull request: [SPARK-15585][SQL] Fix NULL handling along wit...

2016-05-27 Thread maropu
GitHub user maropu opened a pull request: https://github.com/apache/spark/pull/13372 [SPARK-15585][SQL] Fix NULL handling along with a spark-csv behaivour ## What changes were proposed in this pull request? This pr fixes the behaviour of `format("csv").option("quote", null)`

[GitHub] spark pull request: [SPARK-15639][SQL] Try to push down filter at ...

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

[GitHub] spark pull request: [SPARK-15639][SQL] Try to push down filter at ...

2016-05-27 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/13371#issuecomment-90971 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-15639][SQL] Try to push down filter at ...

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

[GitHub] spark pull request: [SPARK-15639][SQL] Try to push down filter at ...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13371#issuecomment-90964 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-15639][SQL] Try to push down filter at ...

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

[GitHub] spark pull request: [SPARK-15639][SQL] Try to push down filter at ...

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

[GitHub] spark pull request: [SPARK-15639][SQL] Try to push down filter at ...

2016-05-27 Thread viirya
GitHub user viirya opened a pull request: https://github.com/apache/spark/pull/13371 [SPARK-15639][SQL] Try to push down filter at RowGroups level for parquet reader ## What changes were proposed in this pull request? When we use vecterized parquet reader, although the

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-27 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/13370#discussion_r64984155 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala --- @@ -41,15 +41,13 @@ import

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-27 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13370#discussion_r64983965 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala --- @@ -41,15 +41,13 @@ import

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-27 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13370#discussion_r64983961 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLContextSuite.scala --- @@ -24,6 +24,7 @@ import org.apache.spark.sql.catalyst.TableIdentifier

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-27 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13370#discussion_r64983958 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileCatalogSuite.scala --- @@ -19,6 +19,8 @@ package

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-27 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13370#discussion_r64983960 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveDDLCommandSuite.scala --- @@ -37,9 +35,9 @@ class HiveDDLCommandSuite extends PlanTest {

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-27 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13370#discussion_r64983952 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala --- @@ -52,7 +52,7 @@ case class CatalogStorageFormat(

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-27 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/13370 [SPARK-15638][SQL] Audit Dataset, SparkSession, and SQLContext ## What changes were proposed in this pull request? This patch contains a list of changes as a result of my auditing Dataset,

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

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

[GitHub] spark pull request: [SPARK-15553][SQL] Dataset.createTempView shou...

2016-05-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13327 --- 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-15553][SQL] Dataset.createTempView shou...

2016-05-27 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13327#issuecomment-89291 Given the comments are pretty minor, I'm going to merge this first. I have some minor cleanups and I will include @andrewor14's comment there. --- If your project is

[GitHub] spark pull request: [SPARK-15633][minor] Make package name for Jav...

2016-05-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13364 --- 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-15633][minor] Make package name for Jav...

2016-05-27 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13364#issuecomment-89156 Merging in master/2.0. --- 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-13184][SQL] Add a datasource-specific o...

2016-05-27 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13320#issuecomment-89137 I'm going to mark this for 2.1 given we are too close to 2.0 to modify behaviors like this. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-15633][minor] Make package name for Jav...

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

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-27 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r64983139 --- Diff: R/pkg/R/deserialize.R --- @@ -197,6 +197,31 @@ readMultipleObjects <- function(inputCon) { data # this is a list of named lists now }

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-27 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r64983076 --- Diff: core/src/main/scala/org/apache/spark/api/r/RRunner.scala --- @@ -149,12 +150,23 @@ private[spark] class RRunner[U](

[GitHub] spark pull request: [SPARK-15549][SQL] Disable bucketing when the ...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13321#issuecomment-86829 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-15549][SQL] Disable bucketing when the ...

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

[GitHub] spark pull request: [SPARK-15549][SQL] Disable bucketing when the ...

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

[GitHub] spark pull request: [SPARK-15517][SQL][STREAMING] Add support for ...

2016-05-27 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/13286#issuecomment-86598 @rxin @marmbrus Are you okay with current OutputMode design? Add unit test for Java compatibility of OutputMode. --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-15610][ML] update error message for k i...

2016-05-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13356 --- 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-15610][ML] update error message for k i...

2016-05-27 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/13356#issuecomment-86536 Merged to master/2.0 --- 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-15636][SQL] Make aggregate expressions ...

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

[GitHub] spark pull request: [SPARK-15610][ML] update error message for k i...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13356#issuecomment-86425 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-15610][ML] update error message for k i...

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

[GitHub] spark pull request: [SPARK-15610][ML] update error message for k i...

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

[GitHub] spark pull request: [WIP][SPARK-6320][SQL] Move planLater method i...

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

[GitHub] spark pull request: [WIP][SPARK-6320][SQL] Move planLater method i...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13147#issuecomment-85952 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: [WIP][SPARK-6320][SQL] Move planLater method i...

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

[GitHub] spark pull request: [WIP][SPARK-6320][SQL] Move planLater method i...

2016-05-27 Thread ueshin
Github user ueshin commented on a diff in the pull request: https://github.com/apache/spark/pull/13147#discussion_r64982752 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/QueryPlanner.scala --- @@ -47,17 +55,27 @@ abstract class

[GitHub] spark pull request: [SPARK-15557][SQL] expressi[on ((cast(99 as de...

2016-05-27 Thread dilipbiswal
Github user dilipbiswal commented on a diff in the pull request: https://github.com/apache/spark/pull/13368#discussion_r64982756 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala --- @@ -290,11 +290,6 @@ object TypeCoercion {

[GitHub] spark pull request: [SPARK-15495][SQL]Improve the explain output f...

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

[GitHub] spark pull request: [SPARK-15495][SQL]Improve the explain output f...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13363#issuecomment-85364 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-15495][SQL]Improve the explain output f...

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

[GitHub] spark pull request: [SPARK-15517][SQL][STREAMING] Add support for ...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13286#issuecomment-85060 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-15517][SQL][STREAMING] Add support for ...

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

[GitHub] spark pull request: [SPARK-15517][SQL][STREAMING] Add support for ...

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

[GitHub] spark pull request: [SPARK-15610][ML] update error message for k i...

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

[GitHub] spark pull request: [SPARK-15562][ML] Delete temp directory after ...

2016-05-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13328 --- 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-15562][ML] Delete temp directory after ...

2016-05-27 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/13328#issuecomment-84410 Merged to master/2.0 --- 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-15449][MLlib][Example]:Wrong Data Forma...

2016-05-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13301 --- 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-15449][MLlib][Example]:Wrong Data Forma...

2016-05-27 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/13301#issuecomment-84311 Merged to master/2.0 --- 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-15610][ML] update error message for k i...

2016-05-27 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13356#discussion_r64982204 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/feature/PCA.scala --- @@ -40,8 +40,9 @@ class PCA @Since("1.4.0") (@Since("1.4.0") val k: Int) {

[GitHub] spark pull request: [SPARK-15637][SPARKR] fix R tests on R 3.2.2

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

[GitHub] spark pull request: [SPARK-15637][SPARKR] fix R tests on R 3.2.2

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13369#issuecomment-83693 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-15637][SPARKR] fix R tests on R 3.2.2

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

[GitHub] spark pull request: [SPARK-15549][SQL] Disable bucketing when the ...

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

[GitHub] spark pull request: [SPARK-15549][SQL] Disable bucketing when the ...

2016-05-27 Thread watermen
Github user watermen commented on the pull request: https://github.com/apache/spark/pull/13321#issuecomment-83360 @andrewor14 @cloud-fan Thanks for your advice, please review the code again. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [WIP][SPARK-6320][SQL] Move planLater method i...

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

[GitHub] spark pull request: [SPARK-15636][SQL] Make aggregate expressions ...

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

[GitHub] spark pull request: [SPARK-15557][SQL] expressi[on ((cast(99 as de...

2016-05-27 Thread dilipbiswal
Github user dilipbiswal commented on a diff in the pull request: https://github.com/apache/spark/pull/13368#discussion_r64981737 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala --- @@ -290,11 +290,6 @@ object TypeCoercion {

[GitHub] spark pull request: [WIP][SPARK-6320][SQL] Move planLater method i...

2016-05-27 Thread ueshin
Github user ueshin commented on a diff in the pull request: https://github.com/apache/spark/pull/13147#discussion_r64981719 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/QueryPlanner.scala --- @@ -47,17 +55,27 @@ abstract class

[GitHub] spark pull request: [SPARK-15637][SPARKR] fix R tests on R 3.2.2

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

[GitHub] spark pull request: [SPARK-15637][SPARKR] fix R tests on R 3.2.2

2016-05-27 Thread felixcheung
GitHub user felixcheung opened a pull request: https://github.com/apache/spark/pull/13369 [SPARK-15637][SPARKR] fix R tests on R 3.2.2 ## What changes were proposed in this pull request? Change version check in R tests ## How was this patch tested? R

[GitHub] spark pull request: [SPARK-15636][SQL] Make aggregate expressions ...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13367#issuecomment-82812 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-15636][SQL] Make aggregate expressions ...

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

[GitHub] spark pull request: [SPARK-15636][SQL] Make aggregate expressions ...

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

[GitHub] spark pull request: [SPARK-15633][minor] Make package name for Jav...

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

[GitHub] spark pull request: [SPARK-15636][SQL] Make aggregate expressions ...

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

[GitHub] spark pull request: [SPARK-15557][SQL] expressi[on ((cast(99 as de...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13368#issuecomment-82415 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-15636][SQL] Make aggregate expressions ...

2016-05-27 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13367#issuecomment-82403 @cloud-fan updated. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-15557][SQL] expressi[on ((cast(99 as de...

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

[GitHub] spark pull request: [SPARK-15557][SQL] expressi[on ((cast(99 as de...

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

[GitHub] spark pull request: [SPARK-15495][SQL]Improve the explain output f...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13363#issuecomment-82263 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-15495][SQL]Improve the explain output f...

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

[GitHub] spark pull request: [SPARK-15495][SQL]Improve the explain output f...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13363#issuecomment-82203 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-15495][SQL]Improve the explain output f...

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

[GitHub] spark pull request: [SPARK-15495][SQL]Improve the explain output f...

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

[GitHub] spark pull request: [SPARK-15636][SQL] Make aggregate expressions ...

2016-05-27 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13367#discussion_r64981440 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala --- @@ -126,7 +137,9 @@ private[sql] case class

[GitHub] spark pull request: [SPARK-15495][SQL]Improve the explain output f...

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

[GitHub] spark pull request: [SPARK-15495][SQL]Improve the explain output f...

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

[GitHub] spark pull request: [SPARK-15495][SQL]Improve the explain output f...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13363#issuecomment-81471 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-15495][SQL]Improve the explain output f...

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

[GitHub] spark pull request: [SPARK-15517][SQL][STREAMING] Add support for ...

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

[GitHub] spark pull request: [SPARK-15610][ML] update error message for k i...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13356#issuecomment-80836 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-15610][ML] update error message for k i...

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

[GitHub] spark pull request: [SPARK-15517][SQL][STREAMING] Add support for ...

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

[GitHub] spark pull request: [SPARK-15610][ML] update error message for k i...

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

[GitHub] spark pull request: [SPARK-15517][SQL][STREAMING] Add support for ...

2016-05-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13286#issuecomment-80806 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-15517][SQL][STREAMING] Add support for ...

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

[GitHub] spark pull request: [SPARK-15517][SQL][STREAMING] Add support for ...

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

[GitHub] spark pull request: [SPARK-15495][SQL]Improve the explain output f...

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

[GitHub] spark pull request: [SPARK-15594][SQL] ALTER TABLE SERDEPROPERTIES...

2016-05-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13343 --- 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-15517][SQL][STREAMING] Add support for ...

2016-05-27 Thread tdas
Github user tdas commented on a diff in the pull request: https://github.com/apache/spark/pull/13286#discussion_r64980368 --- Diff: sql/catalyst/src/main/java/org/apache/spark/sql/OutputMode.java --- @@ -0,0 +1,54 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

  1   2   3   4   5   6   7   8   >