This is an automated email from the ASF dual-hosted git repository.

rui pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-gluten.git


The following commit(s) were added to refs/heads/main by this push:
     new f66481e1af [GLUTEN-7440][VL] Enable unit tests on missing all struct 
fields (#7456)
f66481e1af is described below

commit f66481e1aff0b11ac2d1c7d5539b50e9a81c9f66
Author: Rui Mo <[email protected]>
AuthorDate: Thu Oct 10 16:58:22 2024 +0800

    [GLUTEN-7440][VL] Enable unit tests on missing all struct fields (#7456)
---
 ep/build-velox/src/get_velox.sh                                         | 2 +-
 .../test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala    | 2 --
 .../test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala    | 2 --
 .../test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala    | 2 --
 4 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/ep/build-velox/src/get_velox.sh b/ep/build-velox/src/get_velox.sh
index 03162dc588..f5bb6b3180 100755
--- a/ep/build-velox/src/get_velox.sh
+++ b/ep/build-velox/src/get_velox.sh
@@ -17,7 +17,7 @@
 set -exu
 
 VELOX_REPO=https://github.com/oap-project/velox.git
-VELOX_BRANCH=2024_10_09
+VELOX_BRANCH=2024_10_09_fix
 VELOX_HOME=""
 
 OS=`uname -s`
diff --git 
a/gluten-ut/spark33/src/test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala
 
b/gluten-ut/spark33/src/test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala
index 49a8c13d27..caa91891cf 100644
--- 
a/gluten-ut/spark33/src/test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala
+++ 
b/gluten-ut/spark33/src/test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala
@@ -704,8 +704,6 @@ class VeloxTestSettings extends BackendTestSettings {
   enableSuite[GlutenParquetInteroperabilitySuite]
     .exclude("parquet timestamp conversion")
   enableSuite[GlutenParquetIOSuite]
-    // TODO:(yuan)
-    .exclude("vectorized reader: missing all struct fields")
     // Exception.
     .exclude("SPARK-35640: read binary as timestamp should throw schema 
incompatible error")
     // Exception msg.
diff --git 
a/gluten-ut/spark34/src/test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala
 
b/gluten-ut/spark34/src/test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala
index 7d76f1c0fa..963fb79a35 100644
--- 
a/gluten-ut/spark34/src/test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala
+++ 
b/gluten-ut/spark34/src/test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala
@@ -685,8 +685,6 @@ class VeloxTestSettings extends BackendTestSettings {
   enableSuite[GlutenParquetInteroperabilitySuite]
     .exclude("parquet timestamp conversion")
   enableSuite[GlutenParquetIOSuite]
-    // TODO:(yuan)
-    .exclude("vectorized reader: missing all struct fields")
     // Velox doesn't write file metadata into parquet file.
     .exclude("Write Spark version into Parquet metadata")
     // Exception.
diff --git 
a/gluten-ut/spark35/src/test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala
 
b/gluten-ut/spark35/src/test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala
index f183b0006b..ed4939595b 100644
--- 
a/gluten-ut/spark35/src/test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala
+++ 
b/gluten-ut/spark35/src/test/scala/org/apache/gluten/utils/velox/VeloxTestSettings.scala
@@ -690,8 +690,6 @@ class VeloxTestSettings extends BackendTestSettings {
   enableSuite[GlutenParquetInteroperabilitySuite]
     .exclude("parquet timestamp conversion")
   enableSuite[GlutenParquetIOSuite]
-    // TODO:(yuan)
-    .exclude("vectorized reader: missing all struct fields")
     // Velox doesn't write file metadata into parquet file.
     .exclude("Write Spark version into Parquet metadata")
     // Exception.


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to