Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10979
---
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 ena
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-177415381
I've merged this in master. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does n
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-177407724
LGTM, thanks for helping fixing this! Will get it merged once I get access
to my laptop.
---
If your project is set up for it, you can reply to this email and have y
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-177099376
cc @liancheng
---
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
e
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176724604
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176724606
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176724183
**[Test build #50376 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50376/consoleFull)**
for PR 10979 at commit
[`de137ed`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176692132
**[Test build #50376 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50376/consoleFull)**
for PR 10979 at commit
[`de137ed`](https://gi
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176688755
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 fe
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176682596
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176682597
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176682558
**[Test build #50373 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50373/consoleFull)**
for PR 10979 at commit
[`de137ed`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176675972
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176675975
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176675716
**[Test build #50361 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50361/consoleFull)**
for PR 10979 at commit
[`2e594af`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176675240
**[Test build #50373 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50373/consoleFull)**
for PR 10979 at commit
[`de137ed`](https://gi
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/10979#discussion_r51238223
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRelation.scala
---
@@ -800,9 +800,21 @@ private[sql] object Parqu
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/10979#discussion_r51237908
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRelation.scala
---
@@ -800,9 +800,21 @@ private[sql] object Parq
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10979#issuecomment-176645619
**[Test build #50361 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50361/consoleFull)**
for PR 10979 at commit
[`2e594af`](https://gi
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/10979
[SPARK-13070][SQL] Better error message when Parquet schema merging fails
Make sure we throw better error messages when Parquet schema merging fails.
You can merge this pull request into a Git repos
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176629923
@rxin @liancheng yes, I would love to 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 y
Github user liancheng closed the pull request at:
https://github.com/apache/spark/pull/10972
---
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 liancheng commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176622130
I'm closing this one.
---
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 th
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176618727
@viirya It would be great if you can help since you are pretty familiar
with this part of code :)
---
If your project is set up for it, you can reply to this email a
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/10972#discussion_r51232373
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRelation.scala
---
@@ -799,9 +799,26 @@ private[sql] object Pa
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176617106
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176617110
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176616985
**[Test build #50354 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50354/consoleFull)**
for PR 10972 at commit
[`2e5eddb`](https://g
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176613206
@viirya do you want to submit a pull request to address your issue?
@liancheng is busy with something right now.
---
If your project is set up for it, you can reply to t
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176607674
**[Test build #50354 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50354/consoleFull)**
for PR 10972 at commit
[`2e5eddb`](https://gi
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10972#discussion_r51226584
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRelation.scala
---
@@ -799,9 +799,26 @@ private[sql] object Parquet
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10972#discussion_r51226549
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRelation.scala
---
@@ -799,9 +799,26 @@ private[sql] object Parquet
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176550457
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176550456
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176550281
**[Test build #50327 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50327/consoleFull)**
for PR 10972 at commit
[`1ce61a4`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176546563
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/10972#discussion_r51221351
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRelation.scala
---
@@ -799,9 +799,26 @@ private[sql] object Parqu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176546560
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176546500
**[Test build #50331 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50331/consoleFull)**
for PR 10972 at commit
[`bfe4987`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176532399
**[Test build #50331 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50331/consoleFull)**
for PR 10972 at commit
[`bfe4987`](https://gi
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10972#discussion_r51218281
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/StructType.scala ---
@@ -402,13 +402,13 @@ object StructType extends AbstractDataType {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10972#discussion_r51218082
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRelation.scala
---
@@ -799,9 +799,31 @@ private[sql] object Parquet
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176524791
**[Test build #50327 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50327/consoleFull)**
for PR 10972 at commit
[`1ce61a4`](https://gi
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10972#discussion_r51218005
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRelation.scala
---
@@ -799,9 +799,31 @@ private[sql] object Parquet
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10972#discussion_r51217905
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRelation.scala
---
@@ -799,9 +799,31 @@ private[sql] object Parquet
GitHub user liancheng opened a pull request:
https://github.com/apache/spark/pull/10972
[SPARK-13070][SQL] Better error message when Parquet schema merging fails
Now we also report path and schema of the file in trouble.
You can merge this pull request into a Git repository by runni
46 matches
Mail list logo