[GitHub] [spark] MrDLontheway commented on a diff in pull request #38893: [Spark-40099][SQL] Merge adjacent CaseWhen branches if their values are the same

Thu, 08 Dec 2022 03:26:19 -0800 


MrDLontheway commented on code in PR #38893:
URL: https://github.com/apache/spark/pull/38893#discussion_r1043247947


##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala:
##########
@@ -639,6 +639,26 @@ object RemoveNoopOperators extends Rule[LogicalPlan] {
   }
 }
 
+/**
+ * Merge case condition expression with same value.
+ */
+object MergeConditionWithValue extends Rule[LogicalPlan] {

Review Comment:
   this is rule execution time for 2000 branches 
   <img width="1316" alt="image" 
src="https://user-images.githubusercontent.com/29403543/206434948-6146d550-a1e0-4bc4-b410-27f3bbaf794a.png";>
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to