wForget commented on code in PR #1575:
URL: https://github.com/apache/datafusion-comet/pull/1575#discussion_r2016533206
##########
spark/src/test/scala/org/apache/comet/parquet/ParquetReadSuite.scala:
##########
@@ -1460,6 +1460,25 @@ class ParquetReadV1Suite extends ParquetReadSuite with
AdaptiveSparkPlanHelper {
v1 = Some("parquet"))
}
}
+
+ test("test V1 parquet scan uses native_iceberg_compat -- case insensitive") {
+ withTempPath { path =>
+ spark.range(10).toDF("a").write.parquet(path.toString)
+ Seq(CometConf.SCAN_NATIVE_DATAFUSION,
CometConf.SCAN_NATIVE_ICEBERG_COMPAT).foreach(
+ scanMode => {
+ withSQLConf(CometConf.COMET_NATIVE_SCAN_IMPL.key -> scanMode) {
+ withTable("test") {
+ sql("create table test (A long) using parquet options (path '" +
path + "')")
+ val df = sql("select A from test where A > 5")
+ checkSparkAnswer(df)
+ // TODO: pushed down filters do not used schema adapter in
datafusion, will cause empty result
Review Comment:
This may need improvement in datafusion, possibly relevant code:
https://github.com/apache/datafusion/blob/18feb8b2702b96a8a77ec4bc52fb67571e857d4d/datafusion/datasource-parquet/src/opener.rs#L86
--
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]
