yihua commented on code in PR #9276:
URL: https://github.com/apache/hudi/pull/9276#discussion_r1285164631
##########
hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/LegacyHoodieParquetFileFormat.scala:
##########
@@ -23,12 +23,12 @@ import org.apache.hudi.{DataSourceReadOptions,
HoodieSparkUtils, SparkAdapterSup
import org.apache.spark.sql.SparkSession
import org.apache.spark.sql.catalyst.InternalRow
import org.apache.spark.sql.execution.datasources.PartitionedFile
-import
org.apache.spark.sql.execution.datasources.parquet.HoodieParquetFileFormat.FILE_FORMAT_ID
+import
org.apache.spark.sql.execution.datasources.parquet.LegacyHoodieParquetFileFormat.FILE_FORMAT_ID
import org.apache.spark.sql.sources.Filter
import org.apache.spark.sql.types.{AtomicType, StructType}
-class HoodieParquetFileFormat extends ParquetFileFormat with
SparkAdapterSupport {
+class LegacyHoodieParquetFileFormat extends ParquetFileFormat with
SparkAdapterSupport {
Review Comment:
Yeah, my point is developers changing this class should be aware of the new
file format and not miss changing the new file format causing inconsistency, if
necessary.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
