Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14399
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 project does not have this feature
enabled and wishes so, or
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/14399
Sorry for the late review! LGTM, merging to 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14399
**[Test build #65912 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65912/consoleFull)**
for PR 14399 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14399
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 user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14399
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65912/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14399
**[Test build #65912 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65912/consoleFull)**
for PR 14399 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14399
Hi @liancheng , I'd appreciate if you could take a look 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
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14399
Gentle ping @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
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14399
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63031/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14399
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 user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14399
**[Test build #63031 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63031/consoleFull)**
for PR 14399 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14399
Please let me cc @srowen here as well because it is anyway about building.
---
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 issue:
https://github.com/apache/spark/pull/14399
**[Test build #63031 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63031/consoleFull)**
for PR 14399 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14399
@holdenk Yes, the some tests in `ParquetSchemaSuite`. If I remember
correctly this, they were,
```
Backwards-compatibility: LIST with nullable element type - 2
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14399
This looks good - I might add a comment about why we are using this
approach instead of listOfElements in the code so that whoever comes along next
understands why we couldn't use it.
Also
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14399
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62998/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14399
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 user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14399
**[Test build #62998 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62998/consoleFull)**
for PR 14399 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14399
**[Test build #62998 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62998/consoleFull)**
for PR 14399 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14399
cc @holdenk and @liancheng
Thanks @holdenk, I was stupid and indeed, the alternative of `listType` is
not necessarily `listOfElements` but we can actually work around.
---
If your
20 matches
Mail list logo