[GLUTEN-8010][CORE] Don't generate native metrics if transformer don't generate relNode

gluten-substrait/src/main/scala/org/apache/gluten/execution/BasicPhysicalOperatorTransformer.scala

@@ -61,7 +61,9 @@ abstract class FilterExecTransformerBase(val cond: Expression, val input: SparkP
     case _ => false
   }
 
-  override def metricsUpdater(): MetricsUpdater = if (getRemainingCondition == null) {
+  override def isLoop: Boolean = getRemainingCondition == null
+
+  override def metricsUpdater(): MetricsUpdater = if (isLoop) {
     MetricsUpdater.None
   } else {
     BackendsApiManager.getMetricsApiInstance.genFilterTransformerMetricsUpdater(metrics)
@@ -152,7 +154,7 @@ abstract class FilterExecTransformerBase(val cond: Expression, val input: SparkP
 
   override protected def doTransform(context: SubstraitContext): TransformContext = {
     val childCtx = child.asInstanceOf[TransformSupport].transform(context)
-    if (metricsUpdater == MetricsUpdater.None) {
+    if (isLoop) {
       // The computing for this filter is not needed.
       // Since some columns' nullability will be removed after this filter, we need to update the
       // outputAttributes of child context.

gluten-substrait/src/main/scala/org/apache/gluten/execution/ExpandExecTransformer.scala

@@ -48,7 +48,9 @@ case class ExpandExecTransformer(
     AttributeSet.fromAttributeSets(projections.flatten.map(_.references))
   }
 
-  override def metricsUpdater(): MetricsUpdater = if (projections == null || projections.isEmpty) {
+  override def isLoop: Boolean = projections == null || projections.isEmpty
+
+  override def metricsUpdater(): MetricsUpdater = if (isLoop) {
     MetricsUpdater.None
   } else {
     BackendsApiManager.getMetricsApiInstance.genExpandTransformerMetricsUpdater(metrics)
@@ -115,7 +117,7 @@ case class ExpandExecTransformer(
 
   override protected def doTransform(context: SubstraitContext): TransformContext = {
     val childCtx = child.asInstanceOf[TransformSupport].transform(context)
-    if (metricsUpdater == MetricsUpdater.None) {
+    if (isLoop) {
       // The computing for this Expand is not needed.
       return childCtx
     }

gluten-substrait/src/main/scala/org/apache/gluten/execution/SortExecTransformer.scala

@@ -44,7 +44,9 @@ case class SortExecTransformer(
   @transient override lazy val metrics =
     BackendsApiManager.getMetricsApiInstance.genSortTransformerMetrics(sparkContext)
 
-  override def metricsUpdater(): MetricsUpdater = if (sortOrder == null || sortOrder.isEmpty) {
+  override def isLoop: Boolean = sortOrder == null || sortOrder.isEmpty
+
+  override def metricsUpdater(): MetricsUpdater = if (isLoop) {
     MetricsUpdater.None
   } else {
     BackendsApiManager.getMetricsApiInstance.genSortTransformerMetricsUpdater(metrics)
@@ -106,7 +108,7 @@ case class SortExecTransformer(
 
   override protected def doTransform(context: SubstraitContext): TransformContext = {
     val childCtx = child.asInstanceOf[TransformSupport].transform(context)
-    if (metricsUpdater == MetricsUpdater.None) {
+    if (isLoop) {
       // The computing for this project is not needed.
       return childCtx
     }

gluten-substrait/src/main/scala/org/apache/gluten/execution/WholeStageTransformer.scala

@@ -100,6 +100,9 @@ trait TransformSupport extends GlutenPlan {
         Seq(plan.executeColumnar())
     }
   }
+
+  // When true, it will not generate relNode, nor will it generate native metrics.
+  def isLoop: Boolean = false
 }
 
 trait LeafTransformSupport extends TransformSupport with LeafExecNode {

gluten-substrait/src/main/scala/org/apache/gluten/execution/WindowExecTransformer.scala

@@ -51,9 +51,9 @@ case class WindowExecTransformer(
   @transient override lazy val metrics =
     BackendsApiManager.getMetricsApiInstance.genWindowTransformerMetrics(sparkContext)
 
-  override def metricsUpdater(): MetricsUpdater = if (
-    windowExpression == null || windowExpression.isEmpty
-  ) {
+  override def isLoop: Boolean = windowExpression == null || windowExpression.isEmpty
+
+  override def metricsUpdater(): MetricsUpdater = if (isLoop) {
     MetricsUpdater.None
   } else {
     BackendsApiManager.getMetricsApiInstance.genWindowTransformerMetricsUpdater(metrics)
@@ -182,7 +182,7 @@ case class WindowExecTransformer(
 
   override protected def doTransform(context: SubstraitContext): TransformContext = {
     val childCtx = child.asInstanceOf[TransformSupport].transform(context)
-    if (metricsUpdater == MetricsUpdater.None) {
+    if (isLoop) {
       // The computing for this operator is not needed.
       return childCtx
     }