From 1db3b5f95270ce990ddfc031c63066db89c5f905 Mon Sep 17 00:00:00 2001 From: Angerszhuuuu Date: Fri, 19 Jan 2024 11:25:24 +0800 Subject: [PATCH] [KYUUBI #5997][Improvement] Avoid unnecessary loop of RuleEliminateTypeOf --- .../plugin/spark/authz/rule/RuleEliminateTypeOf.scala | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/rule/RuleEliminateTypeOf.scala b/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/rule/RuleEliminateTypeOf.scala index 0f3ae136c4a..668f45fbd37 100644 --- a/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/rule/RuleEliminateTypeOf.scala +++ b/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/rule/RuleEliminateTypeOf.scala @@ -25,10 +25,8 @@ import org.apache.kyuubi.plugin.spark.authz.rule.expression.TypeOfPlaceHolder object RuleEliminateTypeOf extends Rule[LogicalPlan] { override def apply(plan: LogicalPlan): LogicalPlan = { - plan.transformUp { case p => - p.transformExpressionsUp { - case toph: TypeOfPlaceHolder => TypeOf(toph.expr) - } + plan.transformExpressionsUp { + case toph: TypeOfPlaceHolder => TypeOf(toph.expr) } } }