[GLUTEN-4194][CORE] Clarify fallback reason when columnar filter is d…

…isabled (#4195)
apache · Dec 28, 2023 · 2cea5a3 · 2cea5a3
1 parent 0f25b2a
commit 2cea5a3
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/gluten-core/src/main/scala/io/glutenproject/extension/columnar/TransformHintRule.scala b/gluten-core/src/main/scala/io/glutenproject/extension/columnar/TransformHintRule.scala
@@ -435,8 +435,10 @@ case class AddTransformHintRule() extends Rule[SparkPlan] {
         case plan: FilterExec =>
           val childIsScan = plan.child.isInstanceOf[FileSourceScanExec] ||
             plan.child.isInstanceOf[BatchScanExec]
-          // When scanOnly is enabled, filter after scan will be offloaded.
-          if ((!scanOnly && !enableColumnarFilter) || (scanOnly && !childIsScan)) {
+          if (!enableColumnarFilter) {
+            TransformHints.tagNotTransformable(plan, "columnar Filter is not enabled in FilterExec")
+          } else if (scanOnly && !childIsScan) {
+            // When scanOnly is enabled, filter after scan will be offloaded.
             TransformHints.tagNotTransformable(
               plan,
               "ScanOnly enabled and plan child is not Scan in FilterExec")