Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-221448131
@rxin 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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-221416716
Yes definitely want to be able to read/write empty dfs.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-221415292
Closing this. But could I please ask if it is basically perfered to support
to wrtie and read empty data back for all data sources and others (or maybe
Parquet,
Github user HyukjinKwon closed the pull request at:
https://github.com/apache/spark/pull/13253
---
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-221342965
Yea let's just do it together. 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-221227884
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/13253#issuecomment-221227888
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/13253#issuecomment-221227705
**[Test build #59196 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59196/consoleFull)**
for PR 13253 at commit
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-221223288
I don't mind closing this. I will close if you think so. I can do this
together later with
[SPARK-10216](https://issues.apache.org/jira/browse/SPARK-10216).
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-221207546
**[Test build #59196 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59196/consoleFull)**
for PR 13253 at commit
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-221187678
@rxin No. it has not been fixed.. So, I wanted to add some test codes first
to check writing and reading empty data to make sure this is working first.
The
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-221171864
Did we ever end up fixing https://issues.apache.org/jira/browse/SPARK-10216
after it was reverted?
---
If your project is set up for it, you can reply to this email and
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-221171371
Hi @rxin and @marmbrus,
As you already know a "critical" issue was found here,
[SPARK-15393](https://issues.apache.org/jira/browse/SPARK-15393). So,
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-220834605
Test PASSed.
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/13253#issuecomment-220834603
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/13253#issuecomment-220834547
**[Test build #59103 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59103/consoleFull)**
for PR 13253 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-220831712
**[Test build #59103 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59103/consoleFull)**
for PR 13253 at commit
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-220831607
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/13253#issuecomment-220830739
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/13253#issuecomment-220830740
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/13253#issuecomment-220830732
**[Test build #59102 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59102/consoleFull)**
for PR 13253 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13253#issuecomment-220830411
**[Test build #59102 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59102/consoleFull)**
for PR 13253 at commit
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/13253
[SPARK-15475][SQL] Add tests for writing and reading back empty data for
Parquet, Json and Text data sources
## What changes were proposed in this pull request?
This PR adds the tests
23 matches
Mail list logo