This is an automated email from the ASF dual-hosted git repository.
philo 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 6c39f7b81c [GLUTEN-8151][CORE] Remove supportRangeExec api (#8760)
6c39f7b81c is described below
commit 6c39f7b81c2a63d9ca84208a03c012a1e23bafb4
Author: 李扬 <[email protected]>
AuthorDate: Tue Feb 18 21:19:32 2025 +0800
[GLUTEN-8151][CORE] Remove supportRangeExec api (#8760)
---
.../scala/org/apache/gluten/backendsapi/clickhouse/CHBackend.scala | 2 --
.../main/scala/org/apache/gluten/backendsapi/velox/VeloxBackend.scala | 3 ---
.../main/scala/org/apache/gluten/backendsapi/BackendSettingsApi.scala | 2 --
.../org/apache/gluten/extension/columnar/validator/Validators.scala | 1 -
4 files changed, 8 deletions(-)
diff --git
a/backends-clickhouse/src/main/scala/org/apache/gluten/backendsapi/clickhouse/CHBackend.scala
b/backends-clickhouse/src/main/scala/org/apache/gluten/backendsapi/clickhouse/CHBackend.scala
index 790fb65f90..73e8bce3fe 100644
---
a/backends-clickhouse/src/main/scala/org/apache/gluten/backendsapi/clickhouse/CHBackend.scala
+++
b/backends-clickhouse/src/main/scala/org/apache/gluten/backendsapi/clickhouse/CHBackend.scala
@@ -445,6 +445,4 @@ object CHBackendSettings extends BackendSettingsApi with
Logging {
}
override def supportWindowGroupLimitExec(rankLikeFunction: Expression):
Boolean = true
-
- override def supportRangeExec(): Boolean = true
}
diff --git
a/backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxBackend.scala
b/backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxBackend.scala
index 650d82dfed..99cba7ea2f 100644
---
a/backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxBackend.scala
+++
b/backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxBackend.scala
@@ -578,7 +578,4 @@ object VeloxBackendSettings extends BackendSettingsApi {
override def supportColumnarArrowUdf(): Boolean = true
override def needPreComputeRangeFrameBoundary(): Boolean = true
-
- override def supportRangeExec(): Boolean = true
-
}
diff --git
a/gluten-substrait/src/main/scala/org/apache/gluten/backendsapi/BackendSettingsApi.scala
b/gluten-substrait/src/main/scala/org/apache/gluten/backendsapi/BackendSettingsApi.scala
index ebdb61cf0c..5bedf021c4 100644
---
a/gluten-substrait/src/main/scala/org/apache/gluten/backendsapi/BackendSettingsApi.scala
+++
b/gluten-substrait/src/main/scala/org/apache/gluten/backendsapi/BackendSettingsApi.scala
@@ -152,6 +152,4 @@ trait BackendSettingsApi {
def supportColumnarArrowUdf(): Boolean = false
def needPreComputeRangeFrameBoundary(): Boolean = false
-
- def supportRangeExec(): Boolean = false
}
diff --git
a/gluten-substrait/src/main/scala/org/apache/gluten/extension/columnar/validator/Validators.scala
b/gluten-substrait/src/main/scala/org/apache/gluten/extension/columnar/validator/Validators.scala
index 0e0f5f444b..509d7c02ae 100644
---
a/gluten-substrait/src/main/scala/org/apache/gluten/extension/columnar/validator/Validators.scala
+++
b/gluten-substrait/src/main/scala/org/apache/gluten/extension/columnar/validator/Validators.scala
@@ -136,7 +136,6 @@ 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()
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
