update

apache · Dec 10, 2024 · c4a7368 · c4a7368
1 parent bdf62b7
commit c4a7368
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/gluten-substrait/src/main/scala/org/apache/gluten/execution/BatchScanExecTransformer.scala b/gluten-substrait/src/main/scala/org/apache/gluten/execution/BatchScanExecTransformer.scala
@@ -109,7 +109,9 @@ abstract class BatchScanExecTransformerBase(
             ExpressionConverter.replaceAttributeReference(expr),
             output)
       }
-    case _ => Seq.empty
+    case _ =>
+      logInfo(s"${scan.getClass.toString} does not support push down filters")
+      Seq.empty
   }
 
   def setPushDownFilters(filters: Seq[Expression]): Unit = {

diff --git a/gluten-substrait/src/main/scala/org/apache/gluten/expression/ExpressionConverter.scala b/gluten-substrait/src/main/scala/org/apache/gluten/expression/ExpressionConverter.scala
@@ -58,12 +58,13 @@ object ExpressionConverter extends SQLConfHelper with Logging {
   def canReplaceWithExpressionTransformer(
       expr: Expression,
       attributeSeq: Seq[Attribute]): Boolean = {
-    val expressionsMap = ExpressionMappings.expressionsMap
     try {
-      replaceWithExpressionTransformer0(expr, attributeSeq, expressionsMap)
+      replaceWithExpressionTransformer(expr, attributeSeq)
       true
     } catch {
-      case _: Exception => false
+      case e: Exception =>
+        logInfo(e.getMessage)
+        false
     }
   }