address review comments

kazuyukitanimura · May 15, 2024 · 396d077 · 396d077
1 parent 9a4b605
commit 396d077
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 4 deletions.
diff --git a/common/src/main/spark-4.0/org/apache/comet/shims/ShimFileFormat.scala b/common/src/main/spark-4.0/org/apache/comet/shims/ShimFileFormat.scala
@@ -23,8 +23,6 @@ import org.apache.spark.sql.execution.datasources.FileFormat
 import org.apache.spark.sql.execution.datasources.parquet.ParquetFileFormat
 
 object ShimFileFormat {
-  val ROW_INDEX = ParquetFileFormat.ROW_INDEX
-
   // A name for a temporary column that holds row indexes computed by the file format reader
   // until they can be placed in the _metadata struct.
   val ROW_INDEX_TEMPORARY_COLUMN_NAME = ParquetFileFormat.ROW_INDEX_TEMPORARY_COLUMN_NAME

diff --git a/common/src/main/spark-4.0/org/apache/spark/sql/comet/shims/ShimCometParquetUtils.scala b/common/src/main/spark-4.0/org/apache/spark/sql/comet/shims/ShimCometParquetUtils.scala
@@ -30,8 +30,6 @@ trait ShimCometParquetUtils {
     QueryExecutionErrors.foundDuplicateFieldInFieldIdLookupModeError(requiredId, matchedFields)
   }
 
-  val FIELD_ID_METADATA_KEY = ParquetUtils.FIELD_ID_METADATA_KEY
-
   def hasFieldIds(schema: StructType): Boolean = ParquetUtils.hasFieldIds(schema)
 
   def hasFieldId(field: StructField): Boolean = ParquetUtils.hasFieldId(field)