[GitHub] spark pull request #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet co...

2016-07-30 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/14419#discussion_r72896264 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetCompatibilityTest.scala --- @@ -119,8 +119,19 @@ private[sql]

[GitHub] spark pull request #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet co...

2016-07-30 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/14419#discussion_r72894773 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetCompatibilityTest.scala --- @@ -119,8 +119,19 @@ private[sql]

[GitHub] spark pull request #14419: [SPARK-16814][SQL][WIP] Fix deprecated parquet co...

2016-07-30 Thread holdenk
GitHub user holdenk opened a pull request: https://github.com/apache/spark/pull/14419 [SPARK-16814][SQL][WIP] Fix deprecated parquet constructor usage ## What changes were proposed in this pull request? Replace deprecated ParquetWriter with the new builders ##