davidm-db commented on code in PR #50027: URL: https://github.com/apache/spark/pull/50027#discussion_r1964236008
########## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/SqlScriptingLogicalPlans.scala: ########## @@ -226,7 +226,7 @@ case class IterateStatement(label: String) extends CompoundPlanStatement { * in WHEN branches. * @param elseBody Body that is executed if none of the conditions are met, i.e. ELSE branch. */ -case class CaseStatement( +case class SearchedCaseStatement( Review Comment: nit: update the description comment, to be equivalent of "SIMPLE variant" as below. Also, we might include a small examples to illustrate which is which, I always get confused... -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org