manuzhang commented on code in PR #13786:
URL: https://github.com/apache/iceberg/pull/13786#discussion_r2529915552
##########
spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/SparkSQLProperties.java:
##########
@@ -30,6 +30,15 @@ private SparkSQLProperties() {}
// Controls which Parquet reader implementation to use
public static final String PARQUET_READER_TYPE =
"spark.sql.iceberg.parquet.reader-type";
public static final ParquetReaderType PARQUET_READER_TYPE_DEFAULT =
ParquetReaderType.ICEBERG;
+
+ // Controls the fully qualified class name of the vectorized Parquet reader
factory
+ public static final String PARQUET_VECTORIZED_READER_FACTORY =
+ "spark.sql.iceberg.parquet.vectorized-reader.factory";
+
+ // Comet vectorized reader factory class name
+ public static final String COMET_VECTORIZED_READER_FACTORY_CLASS =
+ "org.apache.iceberg.spark.parquet.CometVectorizedParquetReaderFactory";
Review Comment:
Why does this config have inconsistent naming style with others?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
