[GitHub] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11604 --- 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] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194640449 Thanks - merging this in master. --- 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] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194632226 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194632224 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-13766][SQL] Consistent file extensions ...

2016-03-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194631579 **[Test build #52793 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52793/consoleFull)** for PR 11604 at commit

[GitHub] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194600375 **[Test build #52793 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52793/consoleFull)** for PR 11604 at commit

[GitHub] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194539680 @rxin Sure. 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 not have

[GitHub] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194456804 BTW can you also explain in code why for parquet we are putting compression codec before .parquet? I think the main reason is that tools such as gunzip wouldn't be able

[GitHub] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194456671 This seems reasonable to me. --- 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] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194456567 Sorry, it does have tests :) --- 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] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194456392 +1 on adding tests. --- 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-13766][SQL] Consistent file extensions ...

2016-03-09 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194456245 You should add tests for any behavior change, otherwise someone could break this when refactoring. --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194252142 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-13766][SQL] Consistent file extensions ...

2016-03-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194252148 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194251592 **[Test build #52744 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52744/consoleFull)** for PR 11604 at commit

[GitHub] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194212135 cc @rxin --- 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-13766][SQL] Consistent file extensions ...

2016-03-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11604#issuecomment-194212115 **[Test build #52744 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52744/consoleFull)** for PR 11604 at commit

[GitHub] spark pull request: [SPARK-13766][SQL] Consistent file extensions ...

2016-03-09 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/11604 [SPARK-13766][SQL] Consistent file extensions for files written by internal data sources ## What changes were proposed in this pull request? This PR makes the file extensions (written