Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9660
---
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/9660#issuecomment-156681414
Thanks! Merging to master and branch-1.6.
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156383995
**[Test build #45856 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45856/consoleFull)**
for PR 9660 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156416659
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156416317
**[Test build #45856 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45856/consoleFull)**
for PR 9660 at commit
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156416602
Finally I got all-pass! Pleace review the codes!
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156416662
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/9660#issuecomment-156370010
**[Test build #45830 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45830/consoleFull)**
for PR 9660 at commit
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156381470
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 user liancheng commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156381453
@HyukjinKwon The spell is "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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156370136
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156370141
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156371302
restart 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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156381799
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156381785
Merged build triggered.
---
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 HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/9660#discussion_r44737977
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetIOSuite.scala
---
@@ -91,6 +91,33 @@ class ParquetIOSuite
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156283866
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156283805
Merged build triggered.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156285575
**[Test build #45791 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45791/consoleFull)**
for PR 9660 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156290598
**[Test build #45798 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45798/consoleFull)**
for PR 9660 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156290645
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/9660#issuecomment-156290642
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156287910
**[Test build #45798 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45798/consoleFull)**
for PR 9660 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156287711
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156287736
Merged build started.
---
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 HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/9660
[SPARK-11694][SQL] Parquet logical types are not being tested properly
All the physical types are properly tested at `ParquetIOSuite` but logical
type mapping is not being tested.
You can
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156059610
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156064202
**[Test build #45729 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45729/consoleFull)**
for PR 9660 at commit
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/9660#discussion_r44648028
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetIOSuite.scala
---
@@ -91,6 +91,32 @@ class ParquetIOSuite
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156096458
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156096460
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/9660#issuecomment-156096326
**[Test build #45729 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45729/consoleFull)**
for PR 9660 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/9660#discussion_r44650410
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetIOSuite.scala
---
@@ -91,6 +91,32 @@ class ParquetIOSuite
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156060344
Merged build triggered.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156300287
**[Test build #45807 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45807/consoleFull)**
for PR 9660 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156310091
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156310079
Merged build triggered.
---
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 HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156321461
Also added an import `Collections` at `ParquetIOSuite`.
Does this also compile the old version of all commits in this PR?
---
If your project is set up
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156321516
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156321507
Merged build triggered.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156310599
**[Test build #45812 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45812/consoleFull)**
for PR 9660 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156316861
**[Test build #45817 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45817/consoleFull)**
for PR 9660 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156316876
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156316877
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/9660#discussion_r44655042
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetIOSuite.scala
---
@@ -91,6 +91,33 @@ class ParquetIOSuite
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156100979
**[Test build #45733 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45733/consoleFull)**
for PR 9660 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156099297
Merged build started.
---
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 liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/9660#discussion_r44654662
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetIOSuite.scala
---
@@ -91,6 +91,33 @@ class ParquetIOSuite
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/9660#discussion_r44654698
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetIOSuite.scala
---
@@ -91,6 +91,33 @@ class ParquetIOSuite
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156099268
Merged build triggered.
---
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 HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156296565
hm. I'm not using `FileMetaData` at line 234 in `ParquetIOSuite`.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156306358
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156306251
**[Test build #45807 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45807/consoleFull)**
for PR 9660 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156306363
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156309901
I added an import `FileMetaData` at ParquetIOSuite.
This is weird, I am suing scala 2.10 and it compiles okay in my local
computer.
---
If your project is set
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156313027
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/9660#issuecomment-156313026
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 user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156296578
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156296698
Merged build triggered.
---
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 HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156313536
Also added an import `ParquetMetadata` at `ParquetIOSuite`
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156314698
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156314688
Merged build triggered.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156314757
**[Test build #45817 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45817/consoleFull)**
for PR 9660 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156321729
**[Test build #45830 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45830/consoleFull)**
for PR 9660 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156296731
Merged build started.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156140292
**[Test build #45733 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45733/consoleFull)**
for PR 9660 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156140509
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156140512
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/9660#issuecomment-156285918
**[Test build #45791 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45791/consoleFull)**
for PR 9660 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156285921
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9660#issuecomment-156285922
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
71 matches
Mail list logo
