Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19707994
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/GeneratedAggregate.scala
---
@@ -70,16 +70,29 @@ case class GeneratedAggregate(
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19708002
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/GeneratedAggregate.scala
---
@@ -70,16 +70,29 @@ case class GeneratedAggregate(
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61386344
[Test build #22705 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22705/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61388496
[Test build #22705 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22705/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61388498
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61388595
[Test build #504 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/504/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61390927
[Test build #504 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/504/consoleFull)
for PR 2983 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/2983
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61226735
[Test build #22605 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22605/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61232397
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61232391
[Test build #22605 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22605/consoleFull)
for PR 2983 at commit
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61296754
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
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19680604
--- Diff: python/pyspark/sql.py ---
@@ -148,13 +149,35 @@ class TimestampType(PrimitiveType):
-class DecimalType(PrimitiveType):
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19680835
--- Diff: python/pyspark/sql.py ---
@@ -439,9 +462,17 @@ def _parse_datatype_json_string(json_string):
return
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61297780
[Test build #22615 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22615/consoleFull)
for PR 2983 at commit
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61305070
Thanks @daveis for the comments; I've now fixed those.
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61306142
[Test build #22617 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22617/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61306356
[Test build #22617 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22617/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61306363
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61307912
[Test build #22620 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22620/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61311521
[Test build #22615 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22615/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61311537
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61323385
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19697168
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/types/dataTypes.scala
---
@@ -90,11 +91,17 @@ object DataType {
| LongType
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19697295
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/parquet/ParquetQuerySuite.scala ---
@@ -869,4 +871,35 @@ class ParquetQuerySuite extends QueryTest
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19697433
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregates.scala
---
@@ -286,18 +286,38 @@ case class
Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19697736
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/types/dataTypes.scala
---
@@ -90,11 +91,17 @@ object DataType {
| LongType ^^^
Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19697753
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/parquet/ParquetQuerySuite.scala ---
@@ -869,4 +871,35 @@ class ParquetQuerySuite extends QueryTest with
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19698052
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/GeneratedAggregate.scala
---
@@ -70,16 +70,29 @@ case class GeneratedAggregate(
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19698090
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/GeneratedAggregate.scala
---
@@ -70,16 +70,29 @@ case class GeneratedAggregate(
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19699927
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala
---
@@ -151,6 +164,7 @@ trait HiveTypeCoercion {
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19699984
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala
---
@@ -250,27 +252,46 @@ case class Cast(child: Expression,
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/2983#discussion_r19700036
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/parquet/ParquetTableSupport.scala
---
@@ -204,6 +205,8 @@ private[parquet] class RowWriteSupport
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61348621
Hey Matei, this is pretty awesome! A few minor comments and this needs to
be merged. Otherwise LGTM.
---
If your project is set up for it, you can reply to this
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61198286
[Test build #22583 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22583/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61207042
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61207038
[Test build #22583 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22583/consoleFull)
for PR 2983 at commit
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60884047
Jenkins, test this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60884309
[Test build #22443 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22443/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60887244
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60887240
[Test build #22443 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22443/consoleFull)
for PR 2983 at commit
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60979400
Jenkins, test this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60979905
[Test build #22459 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22459/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60986161
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60986155
[Test build #22459 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22459/consoleFull)
for PR 2983 at commit
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61008300
Jenkins, test this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61016830
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61017699
[Test build #22484 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22484/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61018102
[Test build #22484 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22484/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61018105
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61020192
Updated this to also read / write decimal precision data from the Hive 0.13
metastore, and fixed a compile error introduced by other recent patches.
---
If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61020575
[Test build #22486 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22486/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61020961
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61020958
[Test build #22486 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22486/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61025985
[Test build #22493 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22493/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61026683
[Test build #22493 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22493/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61026689
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61029320
[Test build #22500 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22500/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61032345
[Test build #22500 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22500/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61032354
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61039162
[Test build #22511 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22511/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61039237
[Test build #22511 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22511/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61039238
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61042108
[Test build #22514 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22514/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61046798
[Test build #22514 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22514/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-61046800
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60859445
I've marked this as not WIP anymore, because the main TODOs left are in the
Hive support. I intend to send that as a separate patch, though I can also add
it here. Right
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60859753
[Test build #22409 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22409/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60861486
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60861481
[Test build #22406 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22406/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60862371
[Test build #22409 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22409/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60862374
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60862637
Jenkins, test this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60862892
[Test build #22414 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22414/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60865367
[Test build #22414 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22414/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60865371
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60866175
[Test build #22418 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22418/consoleFull)
for PR 2983 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60868804
[Test build #22418 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22418/consoleFull)
for PR 2983 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2983#issuecomment-60868809
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
79 matches
Mail list logo