[GitHub] spark issue #13471: [SPARK-15734][SQL] Avoids printing internal row in expla...

2016-06-02 Thread clockfly
Github user clockfly commented on the issue: https://github.com/apache/spark/pull/13471 cc @cloud-fan --- 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

[GitHub] spark pull request #13472: [SPARK-15735] Allow specifying min time to run in...

2016-06-02 Thread ericl
GitHub user ericl opened a pull request: https://github.com/apache/spark/pull/13472 [SPARK-15735] Allow specifying min time to run in microbenchmarks ## What changes were proposed in this pull request? This makes microbenchmarks run for at least 2 seconds by default, to

[GitHub] spark pull request #13446: [SPARK-15704] [SQL] fix a crash in TangstenAggreg...

2016-06-02 Thread inouehrs
Github user inouehrs commented on a diff in the pull request: https://github.com/apache/spark/pull/13446#discussion_r65604802 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TypedAggregateExpression.scala --- @@ -34,6 +34,11 @@ object

[GitHub] spark pull request #13470: [SPARK-15733][SQL] Makes the explain output less ...

2016-06-02 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13470#discussion_r65604769 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -427,13 +427,21 @@ abstract class TreeNode[BaseType <:

[GitHub] spark issue #13077: [SPARK-10748] [Mesos] Log error instead of crashing Spar...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13077 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 does not have this feature

[GitHub] spark pull request #13470: [SPARK-15733][SQL] Makes the explain output less ...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13470#discussion_r65608399 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -427,13 +427,21 @@ abstract class TreeNode[BaseType <:

[GitHub] spark issue #12836: [SPARK-12922][SparkR][WIP] Implement gapply() on DataFra...

2016-06-02 Thread NarineK
Github user NarineK commented on the issue: https://github.com/apache/spark/pull/12836 Thanks, sun-rui. Will look at it. --- 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

[GitHub] spark issue #13051: [SPARK-15271] [MESOS] Allow force pulling executor docke...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13051 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 does not have this feature

[GitHub] spark issue #13051: [SPARK-15271] [MESOS] Allow force pulling executor docke...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13051 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59864/ Test PASSed. ---

[GitHub] spark pull request #13470: [SPARK-15733][SQL] Makes the explain output less ...

2016-06-02 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/spark/pull/13470#discussion_r65610587 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -427,13 +427,21 @@ abstract class TreeNode[BaseType <:

[GitHub] spark issue #13453: [SPARK-15715][SQL] Fix alter partition with storage info...

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

[GitHub] spark issue #13453: [SPARK-15715][SQL] Fix alter partition with storage info...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/13453 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 this feature enabled and wishes

[GitHub] spark pull request #13463: [SPARK-14959][SQL] handle partitioned table direc...

2016-06-02 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13463#discussion_r65618152 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/ListingFileCatalog.scala --- @@ -84,39 +84,12 @@ class ListingFileCatalog(

[GitHub] spark pull request #13463: [SPARK-14959][SQL] handle partitioned table direc...

2016-06-02 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13463#discussion_r65618083 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/ListingFileCatalog.scala --- @@ -84,39 +84,12 @@ class ListingFileCatalog(

[GitHub] spark issue #13407: [SPARK-15665] [CORE] spark-submit --kill and --status ar...

2016-06-02 Thread vanzin
Github user vanzin commented on the issue: https://github.com/apache/spark/pull/13407 I think these tests are flaky, but... 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

[GitHub] spark issue #13451: [SPARK-15711][SQL] Ban CREATE TEMPORARY TABLE USING AS S...

2016-06-02 Thread andrewor14
Github user andrewor14 commented on the issue: https://github.com/apache/spark/pull/13451 LGTM! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] spark issue #13474: [SPARK-15547][SQL] nested case class in encoder can have...

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

[GitHub] spark pull request #13476: [SPARK-15684][SparkR]Not mask startsWith and ends...

2016-06-02 Thread wangmiao1981
GitHub user wangmiao1981 opened a pull request: https://github.com/apache/spark/pull/13476 [SPARK-15684][SparkR]Not mask startsWith and endsWith in R ## What changes were proposed in this pull request? In R 3.3.0, startsWith and endsWith are added. In this PR, I make the

[GitHub] spark pull request #13318: [SPARK-15391] [SQL] manage the temporary memory o...

2016-06-02 Thread sameeragarwal
Github user sameeragarwal commented on a diff in the pull request: https://github.com/apache/spark/pull/13318#discussion_r65628155 --- Diff: core/src/main/java/org/apache/spark/shuffle/sort/ShuffleInMemorySorter.java --- @@ -70,9 +70,14 @@ public int compare(PackedRecordPointer

[GitHub] spark pull request #13456: [SPARK-15724] Add benchmarks for performance over...

2016-06-02 Thread ericl
Github user ericl commented on a diff in the pull request: https://github.com/apache/spark/pull/13456#discussion_r65602557 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/WideSchemaBenchmark.scala --- @@ -0,0 +1,285 @@ +/* + * Licensed to the

[GitHub] spark issue #13077: [SPARK-10748] [Mesos] Log error instead of crashing Spar...

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

[GitHub] spark issue #13446: [SPARK-15704] [SQL] fix a crash in TungstenAggregate whi...

2016-06-02 Thread inouehrs
Github user inouehrs commented on the issue: https://github.com/apache/spark/pull/13446 I fixed my typos. --- 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,

[GitHub] spark issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...

2016-06-02 Thread ioana-delaney
Github user ioana-delaney commented on the issue: https://github.com/apache/spark/pull/13418 @cloud-fan I moved the unit tests to a new test case. Thank you. --- 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 issue #13269: [SPARK-15494][SQL] encoder code cleanup

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13269 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59863/ Test PASSed. ---

[GitHub] spark pull request #13418: [SPARK-15677][SQL] Query with scalar sub-query in...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13418#discussion_r65609442 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SubquerySuite.scala --- @@ -123,6 +123,31 @@ class SubquerySuite extends QueryTest with

[GitHub] spark pull request #13470: [SPARK-15733][SQL] Makes the explain output less ...

2016-06-02 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/spark/pull/13470#discussion_r65611639 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -427,13 +427,21 @@ abstract class TreeNode[BaseType <:

[GitHub] spark pull request #13070: [SPARK-15286] [SQL] Make the output readable for ...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13070#discussion_r65611638 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala --- @@ -65,8 +85,18 @@ case class CatalogColumn( //

[GitHub] spark pull request #13453: [SPARK-15715][SQL] Fix alter partition with stora...

2016-06-02 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/13453#discussion_r65612716 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala --- @@ -779,18 +780,29 @@ private[hive] class HiveClientImpl(

[GitHub] spark issue #13451: [SPARK-15711][SQL] Ban CREATE TEMPORARY TABLE USING AS S...

2016-06-02 Thread liancheng
Github user liancheng commented on the issue: https://github.com/apache/spark/pull/13451 @andrewor14 Could you please help sign off this one? 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 issue #13447: [SPARK-15706] [SQL] Fix Wrong Answer when using IF NOT E...

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

[GitHub] spark issue #13447: [SPARK-15706] [SQL] Fix Wrong Answer when using IF NOT E...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13447 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59865/ Test PASSed. ---

[GitHub] spark issue #13051: [SPARK-15271] [MESOS] Allow force pulling executor docke...

2016-06-02 Thread tnachen
Github user tnachen commented on the issue: https://github.com/apache/spark/pull/13051 @andrewor14 PTAL, this PR LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] spark pull request #13470: [SPARK-15733][SQL] Makes the explain output less ...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13470#discussion_r65622156 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -427,13 +427,20 @@ abstract class TreeNode[BaseType <:

[GitHub] spark issue #13463: [SPARK-14959][SQL] handle partitioned table directories ...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13463 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59885/ Test FAILed. ---

[GitHub] spark issue #13418: [SPARK-15677][SQL] Query with scalar sub-query in the SE...

2016-06-02 Thread ioana-delaney
Github user ioana-delaney commented on the issue: https://github.com/apache/spark/pull/13418 Thank you @cloud-fan. I mentioned the local relations in the test case description and move the test cases under withTempTable. --- If your project is set up for it, you can reply to this

[GitHub] spark issue #12919: [SPARK-15092][SPARK-15139][PYSPARK][ML] Pyspark TreeEnse...

2016-06-02 Thread MLnick
Github user MLnick commented on the issue: https://github.com/apache/spark/pull/12919 jenkins retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] spark issue #13463: [SPARK-14959][SQL] handle partitioned table directories ...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/13463 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 this feature enabled and wishes

[GitHub] spark issue #13463: [SPARK-14959][SQL] handle partitioned table directories ...

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

[GitHub] spark pull request #13477: [SPARK-15739][GraphX] Expose aggregateMessagesWit...

2016-06-02 Thread adeandrade
GitHub user adeandrade opened a pull request: https://github.com/apache/spark/pull/13477 [SPARK-15739][GraphX] Expose aggregateMessagesWithActiveSet to users. ## What changes were proposed in this pull request? It would be fairly trivial to create custom versions of Pregel

[GitHub] spark pull request #13318: [SPARK-15391] [SQL] manage the temporary memory o...

2016-06-02 Thread sameeragarwal
Github user sameeragarwal commented on a diff in the pull request: https://github.com/apache/spark/pull/13318#discussion_r65628414 --- Diff: core/src/main/java/org/apache/spark/shuffle/sort/ShuffleInMemorySorter.java --- @@ -70,9 +70,14 @@ public int compare(PackedRecordPointer

[GitHub] spark issue #13470: [SPARK-15733][SQL] Makes the explain output less verbose...

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

[GitHub] spark issue #13478: [SPARK-15740] [MLLIB] ignore big model load / save in Wo...

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

[GitHub] spark issue #13415: [SPARK-15676] [SQL] Disallow Column Names as Partition C...

2016-06-02 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/13415 Sure, will do it. 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 not have this feature enabled and

[GitHub] spark pull request #13318: [SPARK-15391] [SQL] manage the temporary memory o...

2016-06-02 Thread sameeragarwal
Github user sameeragarwal commented on a diff in the pull request: https://github.com/apache/spark/pull/13318#discussion_r65629505 --- Diff: core/src/main/java/org/apache/spark/shuffle/sort/ShuffleInMemorySorter.java --- @@ -70,9 +70,14 @@ public int compare(PackedRecordPointer

[GitHub] spark issue #13470: [SPARK-15733][SQL] Makes the explain output less verbose...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13470 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59870/ Test PASSed. ---

[GitHub] spark issue #13470: [SPARK-15733][SQL] Makes the explain output less verbose...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13470 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 does not have this feature

[GitHub] spark issue #13318: [SPARK-15391] [SQL] manage the temporary memory of timso...

2016-06-02 Thread sameeragarwal
Github user sameeragarwal commented on the issue: https://github.com/apache/spark/pull/13318 Looks great overall, just few questions and a couple of minor comments. --- 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 issue #13471: [SPARK-15734][SQL] Avoids printing internal row in expla...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/13471 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] spark issue #13143: [SPARK-15359] [Mesos] Mesos dispatcher should handle DRI...

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

[GitHub] spark issue #13469: [SPARK-15140][SQL] make the semantics of null input obje...

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

[GitHub] spark pull request #13446: [SPARK-15704] [SQL] fix a crash in TungstenAggreg...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13446#discussion_r65607282 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TypedAggregateExpression.scala --- @@ -34,6 +34,11 @@ object

[GitHub] spark pull request #13470: [SPARK-15733][SQL] Makes the explain output less ...

2016-06-02 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/spark/pull/13470#discussion_r65607416 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -427,13 +427,21 @@ abstract class TreeNode[BaseType <:

[GitHub] spark pull request #13418: [SPARK-15677][SQL] Query with scalar sub-query in...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13418#discussion_r65609083 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SubquerySuite.scala --- @@ -123,6 +123,31 @@ class SubquerySuite extends QueryTest with

[GitHub] spark pull request #13470: [SPARK-15733][SQL] Makes the explain output less ...

2016-06-02 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/spark/pull/13470#discussion_r65609097 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -427,13 +427,21 @@ abstract class TreeNode[BaseType <:

[GitHub] spark pull request #13470: [SPARK-15733][SQL] Makes the explain output less ...

2016-06-02 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13470#discussion_r65610289 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -427,13 +427,21 @@ abstract class TreeNode[BaseType <:

[GitHub] spark pull request #13463: [SPARK-14959][SQL] handle partitioned table direc...

2016-06-02 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13463#discussion_r65611847 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/ListingFileCatalog.scala --- @@ -84,39 +84,12 @@ class ListingFileCatalog(

[GitHub] spark pull request #13070: [SPARK-15286] [SQL] Make the output readable for ...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13070#discussion_r65611801 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala --- @@ -140,6 +170,32 @@ case class CatalogTable(

[GitHub] spark pull request #13470: [SPARK-15733][SQL] Makes the explain output less ...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13470#discussion_r65612428 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -427,13 +427,21 @@ abstract class TreeNode[BaseType <:

[GitHub] spark issue #13176: [SPARK-15100][DOC] Modified user guide and examples for ...

2016-06-02 Thread GayathriMurali
Github user GayathriMurali commented on the issue: https://github.com/apache/spark/pull/13176 @MLnick Please let me know if there is anything else that I can help with this PR --- 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 #13463: [SPARK-14959][SQL] handle partitioned table direc...

2016-06-02 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13463#discussion_r65612485 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/ListingFileCatalog.scala --- @@ -84,39 +84,12 @@ class ListingFileCatalog(

[GitHub] spark pull request #13415: [SPARK-15676] [SQL] Disallow Column Names as Part...

2016-06-02 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/13415#discussion_r65617955 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -880,6 +880,23 @@ class SparkSqlAstBuilder(conf: SQLConf)

[GitHub] spark pull request #13472: [SPARK-15735] Allow specifying min time to run in...

2016-06-02 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/13472#discussion_r65619857 --- Diff: core/src/main/scala/org/apache/spark/util/Benchmark.scala --- @@ -97,6 +111,39 @@ private[spark] class Benchmark( println //

[GitHub] spark pull request #13470: [SPARK-15733][SQL] Makes the explain output less ...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13470#discussion_r65602110 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -427,13 +427,21 @@ abstract class TreeNode[BaseType <:

[GitHub] spark pull request #13465: [SPARK-15728][SQL] Rename aggregate operators: Ha...

2016-06-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13465 --- 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 issue #13469: [SPARK-15140][SQL] make the semantics of null input obje...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13469 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59860/ Test PASSed. ---

[GitHub] spark issue #13469: [SPARK-15140][SQL] make the semantics of null input obje...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13469 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 does not have this feature

[GitHub] spark issue #13470: [SPARK-15733][SQL] Makes the explain output less verbose...

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

[GitHub] spark pull request #13283: [SPARK-15515] [SQL] Error Handling in Running SQL...

2016-06-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13283 --- 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 #13470: [SPARK-15733][SQL] Makes the explain output less ...

2016-06-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13470#discussion_r65612128 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala --- @@ -427,13 +427,21 @@ abstract class TreeNode[BaseType <:

[GitHub] spark pull request #13463: [SPARK-14959][SQL] handle partitioned table direc...

2016-06-02 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13463#discussion_r65612284 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/ListingFileCatalog.scala --- @@ -84,39 +84,12 @@ class ListingFileCatalog(

[GitHub] spark issue #13447: [SPARK-15706] [SQL] Fix Wrong Answer when using IF NOT E...

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

[GitHub] spark issue #13456: [SPARK-15724] Add benchmarks for performance over wide s...

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

[GitHub] spark pull request #13472: [SPARK-15735] Allow specifying min time to run in...

2016-06-02 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/13472#discussion_r65620010 --- Diff: core/src/main/scala/org/apache/spark/util/Benchmark.scala --- @@ -97,6 +111,39 @@ private[spark] class Benchmark( println //

[GitHub] spark issue #13470: [SPARK-15733][SQL] Makes the explain output less verbose...

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

[GitHub] spark issue #13470: [SPARK-15733][SQL] Makes the explain output less verbose...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13470 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 does not have this feature

[GitHub] spark pull request #13283: [SPARK-15515] [SQL] Error Handling in Running SQL...

2016-06-02 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/13283#discussion_r65624975 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/rules.scala --- @@ -38,6 +40,16 @@ private[sql] class

[GitHub] spark issue #13472: [SPARK-15735] Allow specifying min time to run in microb...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13472 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59869/ Test PASSed. ---

[GitHub] spark issue #13463: [SPARK-14959][SQL] handle partitioned table directories ...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13463 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 does not have this feature

[GitHub] spark issue #13463: [SPARK-14959][SQL] handle partitioned table directories ...

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

[GitHub] spark issue #13463: [SPARK-14959][SQL] handle partitioned table directories ...

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

[GitHub] spark pull request #13454: [SPARK-15714][CORE] Fix flaky o.a.s.scheduler.Bla...

2016-06-02 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/13454#discussion_r65626471 --- Diff: core/src/test/scala/org/apache/spark/scheduler/SchedulerIntegrationSuite.scala --- @@ -307,10 +312,19 @@ private[spark] abstract class

[GitHub] spark pull request #13454: [SPARK-15714][CORE] Fix flaky o.a.s.scheduler.Bla...

2016-06-02 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/13454#discussion_r65626635 --- Diff: core/src/test/scala/org/apache/spark/scheduler/SchedulerIntegrationSuite.scala --- @@ -437,8 +451,8 @@ class BasicSchedulerIntegrationSuite

[GitHub] spark issue #13479: [SPARK-15736][CORE][branch-1.6] Gracefully handle loss o...

2016-06-02 Thread JoshRosen
Github user JoshRosen commented on the issue: https://github.com/apache/spark/pull/13479 /cc @andrewor14, this is the branch-1.6 backport of my other 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 project

[GitHub] spark pull request #13318: [SPARK-15391] [SQL] manage the temporary memory o...

2016-06-02 Thread sameeragarwal
Github user sameeragarwal commented on a diff in the pull request: https://github.com/apache/spark/pull/13318#discussion_r65629261 --- Diff: core/src/main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java --- @@ -221,7 +221,8 @@ public BytesToBytesMap(

[GitHub] spark issue #13473: [SPARK-15736][CORE] Gracefully handle loss of DiskStore ...

2016-06-02 Thread JoshRosen
Github user JoshRosen commented on the issue: https://github.com/apache/spark/pull/13473 @andrewor14, yeah, I think this is the complete set of locations in Spark 2.x. --- 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 issue #13479: [SPARK-15736][CORE][branch-1.6] Gracefully handle loss o...

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

[GitHub] spark issue #13470: [SPARK-15733][SQL] Makes the explain output less verbose...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13470 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59859/ Test PASSed. ---

[GitHub] spark issue #13470: [SPARK-15733][SQL] Makes the explain output less verbose...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13470 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 does not have this feature

[GitHub] spark issue #13407: [SPARK-15665] [CORE] spark-submit --kill and --status ar...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13407 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 does not have this feature

[GitHub] spark issue #13407: [SPARK-15665] [CORE] spark-submit --kill and --status ar...

2016-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13407 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59862/ Test FAILed. ---

[GitHub] spark issue #13258: [SPARK-15472][SQL][Streaming] Add partitioned `csv`, `js...

2016-06-02 Thread zsxwing
Github user zsxwing commented on the issue: https://github.com/apache/spark/pull/13258 @lw-lin could you update this PR soon? Otherwise, we may not be able to merge into 2.0. --- 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 #13471: [SPARK-15734][SQL] Avoids printing internal row i...

2016-06-02 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/spark/pull/13471#discussion_r65612007 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LocalRelation.scala --- @@ -57,7 +57,13 @@ case class

[GitHub] spark pull request #13471: [SPARK-15734][SQL] Avoids printing internal row i...

2016-06-02 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13471#discussion_r65613435 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LocalRelation.scala --- @@ -57,7 +57,13 @@ case class

[GitHub] spark issue #13456: [SPARK-15724] Add benchmarks for performance over wide s...

2016-06-02 Thread ericl
Github user ericl commented on the issue: https://github.com/apache/spark/pull/13456 @rxin updated with parquet variations --- 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 issue #13471: [SPARK-15734][SQL] Avoids printing internal row in expla...

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

[GitHub] spark pull request #13415: [SPARK-15676] [SQL] Disallow Column Names as Part...

2016-06-02 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13415#discussion_r65618427 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -880,6 +880,23 @@ class SparkSqlAstBuilder(conf: SQLConf)

[GitHub] spark issue #13451: [SPARK-15711][SQL] Ban CREATE TEMPORARY TABLE USING AS S...

2016-06-02 Thread andrewor14
Github user andrewor14 commented on the issue: https://github.com/apache/spark/pull/13451 Merging into amster 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 feature enabled and

[GitHub] spark issue #13382: [SPARK-5581][Core] When writing sorted map output file, ...

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

[GitHub] spark pull request #13283: [SPARK-15515] [SQL] Error Handling in Running SQL...

2016-06-02 Thread tedyu
Github user tedyu commented on a diff in the pull request: https://github.com/apache/spark/pull/13283#discussion_r65622737 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/rules.scala --- @@ -38,6 +40,16 @@ private[sql] class

[GitHub] spark issue #13473: [SPARK-15736][CORE] Gracefully handle loss of DiskStore ...

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

  1   2   3   4   5   6   7   8   >