[GitHub] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-24 Thread HyukjinKwon
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-24 Thread rxin
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-24 Thread HyukjinKwon
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-24 Thread HyukjinKwon
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-24 Thread rxin
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-24 Thread AmplabJenkins
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-24 Thread AmplabJenkins
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-24 Thread SparkQA
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-24 Thread HyukjinKwon
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-24 Thread SparkQA
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-24 Thread HyukjinKwon
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-23 Thread rxin
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-23 Thread HyukjinKwon
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-22 Thread AmplabJenkins
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-22 Thread AmplabJenkins
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-22 Thread SparkQA
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-22 Thread SparkQA
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-22 Thread HyukjinKwon
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-22 Thread AmplabJenkins
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-22 Thread AmplabJenkins
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-22 Thread SparkQA
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-22 Thread SparkQA
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] spark pull request: [SPARK-15475][SQL] Add tests for writing and r...

2016-05-22 Thread HyukjinKwon
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