Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20705#discussion_r171668051
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetPartitionDiscoverySuite.scala
---
@@ -739,15 +739,15 @@ class ParquetPartitionDiscoverySuite extends
QueryTest with ParquetTest with Sha
withTempPath { dir =>
df.write.format("parquet").partitionBy(partitionColumns.map(_.name):
_*).save(dir.toString)
val fields = schema.map(f => Column(f.name).cast(f.dataType))
- checkAnswer(spark.read.load(dir.toString).select(fields: _*), row)
+ checkAnswer(spark.read.parquet(dir.toString).select(fields: _*), row)
--- End diff --
Since this is `ParquetPartitionDiscoverySuite`, `parquet` is more proper
than `load`.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
