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]

Reply via email to