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 e7170bede1 [VL] Remove one unnecessary overriding method:
isNullIntolerant (#10086)
e7170bede1 is described below
commit e7170bede12deafcebb8c27e37f55e9df39cc3d9
Author: PHILO-HE <[email protected]>
AuthorDate: Mon Jun 30 16:37:31 2025 +0800
[VL] Remove one unnecessary overriding method: isNullIntolerant (#10086)
---
.../gluten/execution/BasicPhysicalOperatorTransformer.scala | 10 ----------
1 file changed, 10 deletions(-)
diff --git
a/gluten-substrait/src/main/scala/org/apache/gluten/execution/BasicPhysicalOperatorTransformer.scala
b/gluten-substrait/src/main/scala/org/apache/gluten/execution/BasicPhysicalOperatorTransformer.scala
index e1a08cb947..463f540d94 100644
---
a/gluten-substrait/src/main/scala/org/apache/gluten/execution/BasicPhysicalOperatorTransformer.scala
+++
b/gluten-substrait/src/main/scala/org/apache/gluten/execution/BasicPhysicalOperatorTransformer.scala
@@ -55,11 +55,6 @@ abstract class FilterExecTransformerBase(val cond:
Expression, val input: SparkP
protected val notNullAttributes: Seq[ExprId] =
notNullPreds.flatMap(_.references).distinct.map(_.exprId)
- override def isNullIntolerant(expr: Expression): Boolean = expr match {
- case e: NullIntolerant => e.children.forall(isNullIntolerant)
- case _ => false
- }
-
override def isNoop: Boolean = getRemainingCondition == null
override def metricsUpdater(): MetricsUpdater = if (isNoop) {
@@ -191,11 +186,6 @@ abstract class ProjectExecTransformerBase(val list:
Seq[NamedExpression], val in
}()
}
- override def isNullIntolerant(expr: Expression): Boolean = expr match {
- case e: NullIntolerant => e.children.forall(isNullIntolerant)
- case _ => false
- }
-
override def metricsUpdater(): MetricsUpdater =
BackendsApiManager.getMetricsApiInstance.genProjectTransformerMetricsUpdater(metrics)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]