update

apache · Jan 17, 2025 · 7cc9590 · 7cc9590
1 parent 1c926f4
commit 7cc9590
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 12 deletions.
diff --git a/backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxRuleApi.scala b/backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxRuleApi.scala
@@ -19,6 +19,7 @@ package org.apache.gluten.backendsapi.velox
 import org.apache.gluten.GlutenConfig
 import org.apache.gluten.backendsapi.RuleApi
 import org.apache.gluten.columnarbatch.VeloxBatch
+import org.apache.gluten.execution.ColumnarRangeExec
 import org.apache.gluten.extension._
 import org.apache.gluten.extension.columnar._
 import org.apache.gluten.extension.columnar.MiscColumnarRules.{RemoveGlutenTableCacheColumnarToRow, RemoveTopmostColumnarToRow, RewriteSubqueryBroadcast}
@@ -150,7 +151,8 @@ object VeloxRuleApi {
       RasOffload.from[LimitExec](OffloadOthers()),
       RasOffload.from[GenerateExec](OffloadOthers()),
       RasOffload.from[EvalPythonExec](OffloadOthers()),
-      RasOffload.from[SampleExec](OffloadOthers())
+      RasOffload.from[SampleExec](OffloadOthers()),
+      RasOffload.from[ColumnarRangeExec](OffloadOthers())
     )
     offloads.foreach(
       offload =>

diff --git a/gluten-substrait/src/main/scala/org/apache/gluten/execution/RangeExecBaseTransformer.scala b/gluten-substrait/src/main/scala/org/apache/gluten/execution/RangeExecBaseTransformer.scala
@@ -42,17 +42,6 @@ abstract class RangeExecBaseTransformer(
     outputAttributes
   }
 
-  override protected def doValidateInternal(): ValidationResult = {
-    val isSupported = BackendsApiManager.getSettings.supportRangeExec()
-
-    if (!isSupported) {
-      return ValidationResult.failed(
-        s"RangeExec is not supported by the current backend."
-      )
-    }
-    ValidationResult.succeeded
-  }
-
   override def rowType0(): Convention.RowType = Convention.RowType.None
 
   override protected def doExecute()

diff --git a/...-substrait/src/main/scala/org/apache/gluten/extension/columnar/validator/Validators.scala b/...-substrait/src/main/scala/org/apache/gluten/extension/columnar/validator/Validators.scala
@@ -136,6 +136,7 @@ object Validators {
         fail(p)
       case p: CartesianProductExec if !settings.supportCartesianProductExec() => fail(p)
       case p: TakeOrderedAndProjectExec if !settings.supportColumnarShuffleExec() => fail(p)
+      case p: RangeExec if !settings.supportRangeExec() => fail(p)
       case _ => pass()
     }
   }