[GLUTEN-3979][CORE] Use exists() instead of map().exists() to improve…

… code readability
apache · Dec 8, 2023 · 9047cdd · 9047cdd
1 parent 1b87ae9
commit 9047cdd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/gluten-core/src/main/scala/io/glutenproject/extension/StrategyOverrides.scala b/gluten-core/src/main/scala/io/glutenproject/extension/StrategyOverrides.scala
@@ -161,10 +161,10 @@ case class JoinSelectionOverrides(session: SparkSession)
     plan match {
       case plan: Join =>
         if ((count + 1) >= GlutenConfig.getConf.logicalJoinOptimizationThrottle) return true
-        plan.children.map(existsMultiJoins(_, count + 1)).exists(_ == true)
+        plan.children.exists(existsMultiJoins(_, count + 1))
       case plan: Project =>
         if ((count + 1) >= GlutenConfig.getConf.logicalJoinOptimizationThrottle) return true
-        plan.children.map(existsMultiJoins(_, count + 1)).exists(_ == true)
+        plan.children.exists(existsMultiJoins(_, count + 1))
       case other => false
     }
   }

diff --git a/gluten-core/src/main/scala/io/glutenproject/extension/columnar/TransformHintRule.scala b/gluten-core/src/main/scala/io/glutenproject/extension/columnar/TransformHintRule.scala
@@ -157,10 +157,10 @@ case class FallbackMultiCodegens(session: SparkSession) extends Rule[SparkPlan]
     plan match {
       case plan: CodegenSupport if plan.supportCodegen =>
         if ((count + 1) >= optimizeLevel) return true
-        plan.children.map(existsMultiCodegens(_, count + 1)).exists(_ == true)
+        plan.children.exists(existsMultiCodegens(_, count + 1))
       case plan: ShuffledHashJoinExec =>
         if ((count + 1) >= optimizeLevel) return true
-        plan.children.map(existsMultiCodegens(_, count + 1)).exists(_ == true)
+        plan.children.exists(existsMultiCodegens(_, count + 1))
       case other => false
     }