[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-31 Thread holdenk
Github user holdenk commented on the issue: https://github.com/apache/spark/pull/14419 @HyukjinKwon no worries - just wanted to make sure we weren't accidentally duplicating eachothers efforts :) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14419 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63051/ Test PASSed. ---

[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14419 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] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14419 **[Test build #63051 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63051/consoleFull)** for PR 14419 at commit

[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14419 **[Test build #63051 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63051/consoleFull)** for PR 14419 at commit

[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/14419 No, I don't :). I just meant I wanted to say that this fix is reasonable and the reason I didn't submit a PR has not been because there is potential incompatability. It was because just I was

[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread holdenk
Github user holdenk commented on the issue: https://github.com/apache/spark/pull/14419 @HyukjinKwon do you have a PR for this? If so I can go ahead and close this one (and lets maybe sync some on the deprecation stuff we are both cleaning up :)) --- If your project is set up for

[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread holdenk
Github user holdenk commented on the issue: https://github.com/apache/spark/pull/14419 So yes as @HyukjinKwon says its just in the tests. I figured this would be ok across the hadoop versions we target because I looked at the build deps files and they all listed

[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/14419 Could cc @liancheng who has more insight about thie 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

[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/14419 Yes, actually I took a look for this as well and it seems okay (I was thinking doing this with other issues). If my understanding is correct, this would not affect actual writing in Spark

[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14419 Do we know if this works across versions of Parquet that are still in play for Spark builds? I know we've got some YARN and Scala warnings that we can't adjust for that reason. Just double checking.

[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14419 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63039/ Test PASSed. ---

[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14419 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] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14419 **[Test build #63039 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63039/consoleFull)** for PR 14419 at commit

[GitHub] spark issue #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet construct...

2016-07-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14419 **[Test build #63039 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63039/consoleFull)** for PR 14419 at commit