Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15751
I will close this for now and make a new one soon.
---
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 t
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15751
Do we want this change? If there is an approval, I will rebase. It seems
easily making a conflict.
---
If your project is set up for it, you can reply to this email and have your
reply appear o
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15751
Hi @marmbrus, would there be something you are worried of and I should take
a careful look for?
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68326/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68326 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68326/consoleFull)**
for PR 15751 at commit
[`bae8db8`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68326 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68326/consoleFull)**
for PR 15751 at commit
[`bae8db8`](https://github.com/apache/spark/commit/b
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15751
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 wishes s
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68316/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68316 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68316/consoleFull)**
for PR 15751 at commit
[`bae8db8`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68316 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68316/consoleFull)**
for PR 15751 at commit
[`bae8db8`](https://github.com/apache/spark/commit/b
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/15751
Sure, we could possibly merge this first, and you are right, we'd need to
remove checks. I'll try to find some time to look this over as its a larger
patch and I'm focusing on 2.1 bugs right now.
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15751
@marmbrus Sure! makes sense. One thing I instantly worry is, though, IIUC,
it seems we should unset the check[1] for `CalendarIntervalType` for writing
too and I am worried of other side effects
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/15751
Thanks for working on this! My meta question however is why don't we just
support this instead? We can parse interval type from strings and I think it
would be really easy to write a converter th
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68123/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68123 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68123/consoleFull)**
for PR 15751 at commit
[`aea65e9`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68123 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68123/consoleFull)**
for PR 15751 at commit
[`aea65e9`](https://github.com/apache/spark/commit/a
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15751
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 wishes s
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68121/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68121 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68121/consoleFull)**
for PR 15751 at commit
[`aea65e9`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68119/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68119 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68119/consoleFull)**
for PR 15751 at commit
[`9ff247b`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68111/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68111 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68111/consoleFull)**
for PR 15751 at commit
[`d75c11e`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68121 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68121/consoleFull)**
for PR 15751 at commit
[`aea65e9`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68119 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68119/consoleFull)**
for PR 15751 at commit
[`9ff247b`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68105/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68105 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68105/consoleFull)**
for PR 15751 at commit
[`5e87b12`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68104/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68111 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68111/consoleFull)**
for PR 15751 at commit
[`d75c11e`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68104 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68104/consoleFull)**
for PR 15751 at commit
[`370aee0`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68106/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68106 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68106/consoleFull)**
for PR 15751 at commit
[`d247947`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68106 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68106/consoleFull)**
for PR 15751 at commit
[`d247947`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68105 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68105/consoleFull)**
for PR 15751 at commit
[`5e87b12`](https://github.com/apache/spark/commit/5
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15751
cc @rxin and @marmbrus, actual change is only adding
`JacksonUtils.verifySchema(schema)`, `CSVRelation.verifySchema(schema)` and
`TextFileFormat.verifySchema(schema)` and the others are only tes
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15751
Let me try to explain in more details and add some tests for reviewing.
---
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 pr
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68067/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68067 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68067/consoleFull)**
for PR 15751 at commit
[`3836698`](https://github.com/apache/spark/commit/
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15751
cc @marmbrus, could you please take a look? This is the one schema
verification we talked when adding `to_json` .
---
If your project is set up for it, you can reply to this email and have your
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68067 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68067/consoleFull)**
for PR 15751 at commit
[`3836698`](https://github.com/apache/spark/commit/3
50 matches
Mail list logo